Alex, what do you think of using `|` as shortcut separator? I like it (see below for the details)... (In reply to Richard Leger from comment #9) > I am touched! ...and me who thought Alessandro was the UX specialist ;-) Undisputedly so! No worries, I asked for Alex' ui-review on the patch so that will easily override reporter's feedback ;-) That said, we like listening to our users, and while user feedback may have its caveats, it's important nonetheless! > Looks good to me. Your idea - thanks! :-) > Perhaps few suggestions if I may: > - change "Backspace" into "Bksp" ? To make it less looking like an invader in the menu... Yeah, I see what you mean, but "Bksp" would be cryptic and meaningless to most users, so we can't do that. These are localized key translations which I'm retrieving automatically from the system, so we can't even reasonably change that. > - use " | " to replace " / " might be better... "|" is often being interpreted as "OR" keeping space around it... and it may look better... Interesting! Nice and neat. I like it, maybe even better than the slash `/`, because it's calm and less noisy. Users may not know about the OR meaning of `|`, but even if they just take it as a vertical separator, we're still good. Alex, what do you think? > Noticed though that shortcut are missing for Move options... You fox!! Getting those alternative shortcuts to display was a lot harder than expected (if it looks easy in the patch code, that's because it's the n-th iteration after navigating all the problems and optimizing the code), so I figured that I've invested enough of my volunteer time for now ;-) > Those may be the most handy ones considering that drag/drop feature may be delayed to after ESR version possibly... if at least the shortcut could be created and enabled for Move option in ESR version it would be a must! It would ease life :-) Good point! Efficiency is part of our brand core. I'd love that, and I understand from comment 1 that Alex also endorses the idea of adding real keyboard shortcuts for Move-to-* actions. I've suggested some implementation details for consideration at the end of my comment 4. I guess we could do that in a followup bug after first landing what we already have here.
Bug 1647654 Comment 11 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Alex, what do you think of using `|` as shortcut separator? I like it (see below for the details)... (In reply to Richard Leger from comment #9) > I am touched! ...and me who thought Alessandro was the UX specialist ;-) Undisputedly so! No worries, I asked for Alex' ui-review on the patch so that will easily override reporter's feedback ;-) That said, we like listening to our users, and while user feedback may have its caveats, it's important nonetheless! > Looks good to me. Your idea - thanks! :-) > Perhaps few suggestions if I may: > - change "Backspace" into "Bksp" ? To make it less looking like an invader in the menu... Yeah, I see what you mean, but "Bksp" would be cryptic and meaningless to most users, so we can't do that. These are localized key translations which I'm retrieving automatically from the system, so we can't even reasonably change that. > - use " | " to replace " / " might be better... "|" is often being interpreted as "OR" keeping space around it... and it may look better... Interesting! Nice and neat. I like it, maybe even better than the slash `/`, because `|` looks calm and less noisy. Users may not know about the OR meaning of `|`, but even if they just take it as a vertical separator, we're still good. Alex, what do you think? > Noticed though that shortcut are missing for Move options... You fox!! Getting those alternative shortcuts to display was a lot harder than expected (if it looks easy in the patch code, that's because it's the n-th iteration after navigating all the problems and optimizing the code), so I figured that I've invested enough of my volunteer time for now ;-) > Those may be the most handy ones considering that drag/drop feature may be delayed to after ESR version possibly... if at least the shortcut could be created and enabled for Move option in ESR version it would be a must! It would ease life :-) Good point! Efficiency is part of our brand core. I'd love that, and I understand from comment 1 that Alex also endorses the idea of adding real keyboard shortcuts for Move-to-* actions. I've suggested some implementation details for consideration at the end of my comment 4. I guess we could do that in a followup bug after first landing what we already have here.
Alex, what do you think of using `|` as shortcut separator? I like it (see below for the details)... (In reply to Richard Leger from comment #9) > I am touched! ...and me who thought Alessandro was the UX specialist ;-) Undisputedly so! No worries, I asked for Alex' ui-review on the patch so that will easily override reporter's feedback ;-) That said, we like listening to our users, and while user feedback may have its caveats, it's most important nonetheless! > Looks good to me. Your idea - thanks! :-) > Perhaps few suggestions if I may: > - change "Backspace" into "Bksp" ? To make it less looking like an invader in the menu... Yeah, I see what you mean, but "Bksp" would be cryptic and meaningless to most users, so we can't do that. These are localized key translations which I'm retrieving automatically from the system, so we can't even reasonably change that. > - use " | " to replace " / " might be better... "|" is often being interpreted as "OR" keeping space around it... and it may look better... Interesting! Nice and neat. I like it, maybe even better than the slash `/`, because `|` looks calm and less noisy. Users may not know about the OR meaning of `|`, but even if they just take it as a vertical separator, we're still good. Alex, what do you think? > Noticed though that shortcut are missing for Move options... You fox!! Getting those alternative shortcuts to display was a lot harder than expected (if it looks easy in the patch code, that's because it's the n-th iteration after navigating all the problems and optimizing the code), so I figured that I've invested enough of my volunteer time for now ;-) > Those may be the most handy ones considering that drag/drop feature may be delayed to after ESR version possibly... if at least the shortcut could be created and enabled for Move option in ESR version it would be a must! It would ease life :-) Good point! Efficiency is part of our brand core. I'd love that, and I understand from comment 1 that Alex also endorses the idea of adding real keyboard shortcuts for Move-to-* actions. I've suggested some implementation details for consideration at the end of my comment 4. I guess we could do that in a followup bug after first landing what we already have here.
Alex, what do you think of using `|` as shortcut separator? I like it (see below for the details)... (In reply to Richard Leger from comment #9) > I am touched! ...and me who thought Alessandro was the UX specialist ;-) Undisputedly so! No worries, I asked for Alex' ui-review on the patch so that will easily override reporter's feedback ;-) That said, we like listening to our users, and while user feedback may have its caveats, it's most important nonetheless! > Looks good to me. Your idea - thanks! :-) > Perhaps few suggestions if I may: > - change "Backspace" into "Bksp" ? To make it less looking like an invader in the menu... Yeah, I see what you mean, but "Bksp" would be cryptic and meaningless to most users, so we can't do that. These are localized key translations which I'm retrieving automatically from the system, so we can't even reasonably change that. > - use " | " to replace " / " might be better... "|" is often being interpreted as "OR" keeping space around it... and it may look better... Interesting! Nice and neat. I like it, maybe even better than the slash `/`, because `|` looks calm and less noisy. Users may not know about the OR meaning of `|`, but even if they just take it as a vertical separator, we're still good. Alex, what do you think? > Noticed though that shortcut are missing for Move options... You fox!! Getting those alternative shortcuts to display was a lot harder than expected (if it looks easy in the patch code, that's because it's the n-th iteration after navigating all the problems and optimizing the code), so I figured that I've invested enough of my volunteer time for now ;-) > Those may be the most handy ones considering that drag/drop feature may be delayed to after ESR version possibly... if at least the shortcut could be created and enabled for Move option in ESR version it would be a must! It would ease life :-) Good point! Efficiency is part of our brand core. I'd love that, and I understand from comment 1 that Alex also endorses the idea of adding real keyboard shortcuts for Move-to-* actions. I've suggested some implementation details for consideration at the end of my comment 4. I guess we could do that in a followup bug after first landing what we already have here.