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

feat: implement shortcut for opening/closing all CWs #1973

Merged
merged 1 commit into from Feb 28, 2021

Conversation

nolanlawson
Copy link
Owner

Fixes #1914

Adds a keyboard shortcut, z, to open or close all content warnings in a thread. z was chosen because of its proximity to x on a US keyboard, and because Mastodon doesn't seem to have a shortcut for this, or to use z for something else.

This isn't accessible to non-keyboard users, but I consider this a power user feature, and I'm hesitant to clutter up the UI with more buttons, so I consider #1914 closed after this is merged.

@@ -155,6 +155,7 @@ export default {
<li><kbd>p</kbd> das Profil des Verfassers öffnen</li>
<li><kbd>l</kbd> den Link des aktuellen Tröts in einem neuen Tab öffnen</li>
<li><kbd>x</kbd> den Text hinter der Inhaltswarnung anzeigen oder verbergen</li>
<li><kbd>z</kbd> den Text hinter der Inhaltswarnung anzeigen oder verbergen (alle)</li>
Copy link
Owner Author

Choose a reason for hiding this comment

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

I don't speak German but I'm making an effort here. I don't know how to keep translations up-to-date long-term.

@jdaviescoates
Copy link

Is this change already available on pinafore.social ? because nothing happens when I press z (and I'm mostly on my phone and so unable to press z).

@nolanlawson
Copy link
Owner Author

@jdaviescoates It's not yet on pinafore.social. It's on dev.pinafore.social until the next release.

alice-werefox pushed a commit to alice-werefox/sema-werefox-cafe that referenced this pull request Apr 3, 2023
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.

Option to expand content warnings automatically
2 participants