Feature #34549
Add keyboard shortcuts for wiki toolbar buttons
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Go MAEDA | % Done: | 0% | |
| Category: | UI | |||
| Target version: | 4.2.0 | |||
| Resolution: | Fixed |
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!
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch 
(7.96 KB)
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline_v2.patch
(8.67 KB)
tooltips-are-untranslated.png (44.3 KB)
Related issues
Associated revisions
Add keyboard shortcuts for bold, italic and underline buttons (#34549).
Patch by Marius BALTEANU.
Internationalize button titles (#34549).
Patch by Go MAEDA.
History
#1
Updated by Marius BALTEANU about 1 year ago
- Subject changed from Add keyboard shortcuts for wiki toolbar to Add keyboard shortcuts for wiki toolbar buttons
#2
Updated by Marius BALTEANU about 1 year ago
- Description updated (diff)
#3
Updated by Go MAEDA about 1 year 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 about 1 year 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 about 1 year ago
- Blocked by Feature #30459: Switch edit/preview tabs with keyboard shortcuts added
#6
Updated by Marius BALTEANU about 1 year ago
- File deleted (
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch)
#7
Updated by Marius BALTEANU about 1 year ago
- File 0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch added
#8
Updated by Marius BALTEANU about 1 year ago
- File deleted (
0001-Add-keyboard-shortcuts-for-bold-italic-and-underline.patch)
#9
Updated by Marius BALTEANU about 1 year ago
Here is the patch, any feedback is welcome!
#10
Updated by Marius BALTEANU about 1 year ago
- Assignee deleted (
Marius BALTEANU)
#12
Updated by Go MAEDA 12 months 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 12 months 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 12 months ago
Marius BALTEANU wrote:
The new version adds the following changes:
1. Adds a new methodosx?on lib/platform that returns true ifRUBY_PLATFORMis 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.
#16
Updated by Go MAEDA 12 months ago
- File tooltips-are-untranslated.png added
- Status changed from Closed to Reopened
After r20729, tooltips for the buttons always displaying in English regardless of the current user's Language.
The following code fixes the issue.
diff --git a/public/javascripts/jstoolbar/jstoolbar.js b/public/javascripts/jstoolbar/jstoolbar.js
index ed6605965..04c35fa95 100644
--- a/public/javascripts/jstoolbar/jstoolbar.js
+++ b/public/javascripts/jstoolbar/jstoolbar.js
@@ -250,10 +250,16 @@ jsToolBar.prototype = {
return b;
},
buttonTitleWithShortcut: function(title, shortcutKey) {
+ if(typeof jsToolBar.strings == 'undefined') {
+ var translatedTitle = title || null;
+ } else {
+ var translatedTitle = jsToolBar.strings[title] || title || null;
+ }
+
if (isMac) {
- return title + " (⌘" + shortcutKey.toUpperCase() + ")";
+ return translatedTitle + " (⌘" + shortcutKey.toUpperCase() + ")";
} else {
- return title + " (Ctrl+" + shortcutKey.toUpperCase() + ")";
+ return translatedTitle + " (Ctrl+" + shortcutKey.toUpperCase() + ")";
}
},
space: function(toolName) {