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

Option for nested context menu #44

Closed
rugk opened this issue May 19, 2021 · 2 comments · Fixed by #84
Closed

Option for nested context menu #44

rugk opened this issue May 19, 2021 · 2 comments · Fixed by #84
Labels
enhancement New feature or request

Comments

@rugk
Copy link
Owner

rugk commented May 19, 2021

Background

As explained in #25 (comment):

We could always create a nested menu, but that would of course be harder for the user to use and we would then not be able to show all the options in a single screenshot.

#25 (comment):

I’m getting lost in that many options, actually, already…

Proposed solution

I should be able to enable a nested context menu, where the different transformations are somewhat of being grouped together…

Alternatives

Maybe some thing of ”highlight top most used 5 ones” at the top or sort the entries by what was last used etc.
If you’d want that, please open a new issue, however.

Additional context

N/A

@rugk rugk added the enhancement New feature or request label May 19, 2021
@tdulcet
Copy link
Collaborator

tdulcet commented May 20, 2021

OK, I do not think your “Proposed solution” is what you meant, so maybe we could structure the nested context menu something like this:

  • Superscript
  • Small Caps
    • Small Caps
    • All Small Caps
    • Unicase
  • Serif
    • Serif bold
    • Serif italic
    • Serif bold italic
  • Sans-serif
    • Sans-serif
    • Sans-serif bold
    • Sans-serif italic
    • Sans-serif bold italic
  • Script
    • Script
    • Script bold
  • Fraktur
    • Fraktur
    • Fraktur bold
  • Monospace
  • Double-struck
  • Other/Emojis
    • Circled
    • Circled (Black)
    • Squared
    • Squared (Black)
  • Fullwidth
  • Change casing
    • Lowercase
    • Uppercase
    • Capitalize Each Word
    • Toggle Case

I am not sure if you wanted all the items in nested menus or if you wanted more than one level of nesting. I am also not sure how this would work with the live preview option. Anyway, this will likely require a significant refactoring of the text transformation feature, similar to #17.

@rugk
Copy link
Owner Author

rugk commented May 20, 2021

OK, I do not think your “Proposed solution” is what you meant

Oh sorry, I had a syntax error here (cough cough caused by this add-on #14), which commented a bigger part. Corrected now.

am also not sure how this would work with the live preview option.

Hmm, just do not live-preview the categories should be enough, should not it? 🤔
Though it may look strange if some things are previewed and some are not. 🤔
Anyway… taht’s an implementation detail/UX thing that can be found out when implementing it and looking/comparing how it looks. I could also imagine it does sth. like not changing the top-level entries like “Superscript” to a live preview only, but still always forces to add the description, so one knows what is done there.

But thanks for making a list of all options! It’s a good proposal that could be implemented later on…

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 a pull request may close this issue.

2 participants