Skip to content

Implement Class Documentation Preview Webviews #754

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Oct 27, 2021

Conversation

isc-bsaviano
Copy link
Contributor

This PR adds webviews for previewing documentation for the currently open ObjectScript class. The preview will display the rendered HTML description for the class member that the user's cursor is in, or the class itself if the user's cursor isn't inside a class member. If a user switches to a different class, the preview will "follow" the user's cursor and display the documentation for the more recent class. The preview is dynamic (it will update as the user changes the class being previewed) and does not require an active connection to a server to work. The gif below demonstrates the workflow:

DocumaticPreview

@isc-bsaviano isc-bsaviano added the enhancement New feature or request label Oct 25, 2021
daimor
daimor previously approved these changes Oct 25, 2021
Copy link
Contributor

@gjsjohnmurray gjsjohnmurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my suggested change to avoid switching focus.

I also wonder if the webview needs a better name for its tab than "Preview Package.Subpackage.Class.cls". Maybe "Doc for Package.Subpackage.Class".

@isc-bsaviano
Copy link
Contributor Author

@gjsjohnmurray I chose "Preview" because that's the wording that the builtin Markdown extension uses. I'm ok with switching to "Doc for .." if you and @daimor prefer that

@daimor
Copy link
Member

daimor commented Oct 26, 2021

I agree with John, Doc for sounds better
Preview would better suit for CSP.Page based classes.

@isc-bsaviano isc-bsaviano merged commit d40a4f4 into intersystems-community:master Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants