Feature #34549
Add keyboard shortcuts for wiki toolbar buttons
Status: | New | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | UI | |||
Target version: | 4.2.0 | |||
Resolution: |
Description
Add keyboard shortcuts for the following buttons from wiki toolbar:
Action | Keyboard shortcut |
Add bold | Ctrl/⌘ + b |
Add italic | Ctrl/⌘ + i |
Add underline | Ctrl/⌘ + u |
⌘: should work only on osx
Ctrl: should work only on windows/linux.
Any other proposals are welcome!
Related issues
History
#1
Updated by Marius BALTEANU 22 days ago
- Subject changed from Add keyboard shortcuts for wiki toolbar to Add keyboard shortcuts for wiki toolbar buttons
#2
Updated by Marius BALTEANU 22 days ago
- Description updated (diff)
#3
Updated by Go MAEDA 22 days ago
I like the feature. Since the key combination ⌘ + b is available in so many apps, I sometimes mistakenly use the key combination in Redmine.
Ctrl: should work only on windows/linux.
It is really important. Ctrl + B is already used as a shortcut key of the left arrow key in macOS.
#4
Updated by Marius BALTEANU 18 days ago
- File 0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch added
Here is an almost ready version of the patch for testing purposes. Must be applied on top of #30459.
I need to test the feature on multiple browsers/devices, add system tests and improve the code if possible.
#5
Updated by Marius BALTEANU 18 days ago
- Blocked by Feature #30459: Switch edit/preview tabs with keybord shortcuts added
#6
Updated by Marius BALTEANU 18 days ago
- File deleted (
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch)
#7
Updated by Marius BALTEANU 18 days ago
- File 0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch added
#8
Updated by Marius BALTEANU 16 days ago
- File deleted (
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch)
#9
Updated by Marius BALTEANU 16 days ago
Here is the patch, any feedback is welcome!
#10
Updated by Marius BALTEANU 16 days ago
- Assignee deleted (
Marius BALTEANU)
#12
Updated by Go MAEDA 3 days ago
Thank you for writing the patch but I found some tests fail on macOS. This is because the tests use a control key. As written in the description, control key does not work on macOS.
laphroaig:redmine-trunk maeda$ ruby test/system/keyboard_shortcuts_test.rb Run options: --seed 58685 # Running: Capybara starting Puma... * Version 5.1.1 , codename: At Your Service * Min threads: 0, max threads: 4 * Listening on http://127.0.0.1:64266 [Screenshot]: tmp/screenshots/failures_test_keyboard_shortcuts_for_wiki_toolbar_buttons_using_textile.png F Failure: InlineAutocompleteSystemTest#test_keyboard_shortcuts_for_wiki_toolbar_buttons_using_textile [test/system/keyboard_shortcuts_test.rb:78]: Expected: "**" Actual: "" bin/rails test test/system/keyboard_shortcuts_test.rb:72 [Screenshot]: tmp/screenshots/failures_test_keyboard_shortcuts_keys_should_be_shown_in_button_title.png F Failure: InlineAutocompleteSystemTest#test_keyboard_shortcuts_keys_should_be_shown_in_button_title [test/system/keyboard_shortcuts_test.rb:117]: Expected: "Strong (Ctrl+B)" Actual: "Strong (⌘B)" bin/rails test test/system/keyboard_shortcuts_test.rb:112 .[Screenshot]: tmp/screenshots/failures_test_keyboard_shortcuts_for_wiki_toolbar_buttons_using_markdown.png F Failure: InlineAutocompleteSystemTest#test_keyboard_shortcuts_for_wiki_toolbar_buttons_using_markdown [test/system/keyboard_shortcuts_test.rb:98]: Expected: "****" Actual: "" bin/rails test test/system/keyboard_shortcuts_test.rb:92 . Finished in 14.276589s, 0.3502 runs/s, 0.9106 assertions/s. 5 runs, 13 assertions, 3 failures, 0 errors, 0 skips
#13
Updated by Marius BALTEANU about 7 hours ago
Thanks Go Maeda for catching this issue, I had in mind only to pass on Linux instances (I've docker on my local env). Updated a new version of the patch that should pass on OSX, but I didn't test it, please let me know how it goes.
The new version adds the following changes:
1. Adds a new method osx?
on lib/platform that returns true if RUBY_PLATFORM
is darwin.
2. In the tests, the modifier key is returned based on osx?
.
#14
Updated by Go MAEDA about 2 hours ago
Marius BALTEANU wrote:
The new version adds the following changes:
1. Adds a new methodosx?
on lib/platform that returns true ifRUBY_PLATFORM
is darwin.
2. In the tests, the modifier key is returned based onosx?
.
Thank you, it now passes all tests also on macOS. I will commit the patch soon.