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

[Feature]: Add dropdown menu in title view containing link to settings and changelog #288

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

mliao95
Copy link

@mliao95 mliao95 commented Mar 3, 2021

This PR adds a dropdown menu in the extension's title view that contains links to the extension's settings and changelog.

GIF:
extension-more-actions

closes #255

@mliao95 mliao95 requested a review from vidorteg March 3, 2021 00:04
@mliao95 mliao95 added this to the 1.1.5 milestone Mar 3, 2021
Copy link
Member

@joselea joselea left a comment

Choose a reason for hiding this comment

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

LGTM, have a quick question

package.json Show resolved Hide resolved
Copy link
Member

@joselea joselea left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@vidorteg vidorteg left a comment

Choose a reason for hiding this comment

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

I like the change but I'm not sure if the toolbar is the best place for it, as it feels out of place with the other buttons (which are extension commands). I took a look to other extensions and some of them add a "..." menu at the end which triggers a context menu where we could add the changelog (along with maybe other options like "Settings"). similar to this:
image

@mliao95 mliao95 changed the title [Feature]: Adding button to title view to view changelog [Feature]: Add dropdown menu in title view containing link to settings and changelog Mar 11, 2021
@mliao95 mliao95 requested a review from vidorteg March 11, 2021 01:07
Copy link
Contributor

@vidorteg vidorteg left a comment

Choose a reason for hiding this comment

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

Looks incredible @mliao95 !, thanks for the quick turnaround

@mliao95 mliao95 merged commit bc2e705 into master Mar 11, 2021
@vidorteg vidorteg deleted the user/milia/changelog branch January 30, 2024 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable a changelog button on the target list
3 participants