Skip to content
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

[VSCODE] Manifest generator tooltips should provide info instead of opening browser #2629

Closed
nmetulev opened this issue Feb 16, 2022 · 0 comments · Fixed by pwa-builder/pwa-studio#68

Comments

@nmetulev
Copy link
Member

Tell us about your feature idea

The tooltips in the manifest generator open a link in the browser and take the developer outside of vscode. The tooltips should provide basic information and allow the developer to click a link to get more information if needed. Opening the browser for each tooltip introduces a lot of back and forth.

Do you have an implementation or a solution in mind?

We could take a one or two sentence description of the property and use that for the tooltip with a link to the full documentation

Have you considered any alternatives?

Alternatively, an inline flyout could load the documentation in the manifest generator. However, this solution is more work and the solution above can provide a cleaner experience.

@jgw96 jgw96 added this to To do in PWABuilder Sprint via automation Feb 16, 2022
@jgw96 jgw96 moved this from To do to Proposed in PWABuilder Sprint Feb 16, 2022
@joshsach joshsach added 2203 and removed p2 labels Feb 24, 2022
@nmetulev nmetulev removed the 2203 label Mar 10, 2022
@nmetulev nmetulev removed the p3 label Apr 22, 2022
@jgw96 jgw96 added this to the PWA Studio1.0 milestone May 3, 2022
PWABuilder Sprint automation moved this from Proposed to Done May 4, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jun 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done ✔️
Development

Successfully merging a pull request may close this issue.

3 participants