marc0l92 / joplin-plugin-jira-issue

Retrieve Atlassian Jira issues information using their api in order to track the status of them from your Joplin notes.
MIT License
27 stars 9 forks source link

Cleaning plugin name from its own submenu #13

Closed graphit0 closed 2 years ago

graphit0 commented 2 years ago

I believe it is obvious that options in menu "Jiraissue" relate to the options of the plugin, hence no need for the text "Jiraissue:" in front of every option in the menu. I believe for accessibility sake making the lines a bit less verbose is beneficial for easier comprehension.

image

marc0l92 commented 2 years ago

Thanks for the pull request. I wrote the prefix JiraIssue: to each command in order to make it more readable in the context menu.

The label of the tools menu is the same of the context menu. In the tools menu, as you pointed out, the prefix is useless, but in the context menu I feel it is a nice to have.

Can you send me a screenshot of the context menu after your pull request?

Maybe one way to have different labels is to duplicate the commands in order to have different labels.

What do you think?

graphit0 commented 2 years ago

image

Well, that was rushed and naive on my part. I agree that the labels would need to be duplicated yet it is larger "scope" than this tiny edit, so I'll abandon this PR.

Regardless, thank you for looking into it and writing a response.