downforacross / downforacross.com

Web frontend for downforacross.com -- continuation of stevenhao/crosswordsio
https://downforacrosscom.downforacross1.now.sh
MIT License
233 stars 91 forks source link

Vim Mode: Enable with keyboard shortcut #264

Closed bkimmig closed 1 year ago

bkimmig commented 1 year ago

What

Add "extras" menu drop down with "vim mode".

Selecting the drop down gives you the opportunity to "Enable Vim Mode"

enable-vim-mode

Once enabled the dropdown gives you the choice to "Disable Vim Mode"

disable-vim-mode
First Proposal - Click to Expand **See discussion below for reasons we did not do this** Add a keyboard shortcut to enable/disable the existing `vimMode`. - Used `:` as the hot key + felt right in the framing of vim + requires a user to press two keys to enable - less likely to happen by accident - Added status to the info tool bar so you can see the state - Can toggle on/off regardless of which mode you are in ## Vim Disabled vim-disabled ## Vim Enabled vim-enabled
vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
downforacross.com ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 25, 2023 at 9:09PM (UTC)
stevenhao commented 1 year ago

hmm, i feel like it could still be pressed on accident. also, : doesn't seem like the right toggle since it's more of a thing you do once you're in vim already?

i don't have any good suggestions on what would be a good hotkey though... maybe a button under a menu somewhere.

bkimmig commented 1 year ago

i feel like it could still be pressed on accident.

Yeah, you are totally correct here after I play tested a bit more.

i don't have any good suggestions on what would be a good hotkey though... maybe a button under a menu somewhere.

I figured since it was removed from the menu it shouldn't live there, but if you'd be ok w/ it in the menu I'd be happy to add it somewhere in there.

Alternatively we could make the hot key combo more complex so you need like 3 keys or more to avoid accidental toggling? ctrl-alt-V or something in that sort of space?

However, if you are more of the mind that "this isn't a feature we want" that is fine and I can move on :).

stevenhao commented 1 year ago

However, if you are more of the mind that "this isn't a feature we want" that is fine and I can move on :).

Personally, I like vim mode enough that i want it to exist and be somewhat discoverable (not necessarily on their first experience -- maybe on a user's 10-50th game or something). i don't personally use it (though i do use vim for programming) but i know there are users (e.g. in the discord) who do use it.

I moved it out of the "Modes Dropdown Menu" as part of a redesign of the toolbar (moving more towards icons & making color attribution its own menu button).

For vim mode, and other "niche features" Perhaps we should add a button to the toolbar that, when clicked, drops down a menu of "Other stuff", where we hide all the uncommonly used features :)

bkimmig commented 1 year ago

For vim mode, and other "niche features" Perhaps we should add a button to the toolbar that, when clicked, drops down a menu of "Other stuff", where we hide all the uncommonly used features :)

Sounds good to me. I'll close out this request and start that as a feature if that is ok?

stevenhao commented 1 year ago

sure, or repurpose this PR to keep discussion in one place. i don't mind either way.

bkimmig commented 1 year ago

@stevenhao I've updated the PR description (collapsed the old one, but kept it). What do you think of this solution?

stevenhao commented 1 year ago

OK! I may want to change this to an icon in the future, for the sake of saving toolbar space. Down to merge this as-is for now. Thanks for working on this!

stevenhao commented 1 year ago

I might also add Color Attribution Mode, List View Mode, Autocheck here too to improve their discoverability. They already have icons in the toolbar, but it's not always obvious how to use them at first glance.