Bug 124177 - Non working menu entry - Help > Show shortcuts
Summary: Non working menu entry - Help > Show shortcuts
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: iOS Editor (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other iOS
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:6.3.0
Keywords:
Depends on:
Blocks:
 
Reported: 2019-03-18 15:51 UTC by Nicolas Christener
Modified: 2019-04-05 18:16 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Tentative patch to hide them (2.45 KB, application/mbox)
2019-04-05 16:04 UTC, How can I remove my account?
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Christener 2019-03-18 15:51:57 UTC
Description:
The dialog containing the shortcuts won't show up in the iOS app.

I'm not sure if this dialog is actually useful on iOS.

Steps to Reproduce:
1. Create/open a document in the iOS app
2. Click the menu entry "Help > Shortcuts"

Actual Results:
Nothing happens.

Expected Results:
A dialog with the shortcuts should show up or the menu entry should be removed.


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 Aron Budea 2019-03-18 20:28:55 UTC
I'd say, would be best to remove for now. I'm not sure what the possibilities are with an external keyboard.
Comment 2 Andras Timar 2019-03-30 15:51:12 UTC
The About dialog is displayed (same type). So there must be a difference, for example I can imagine that loleaflet-help.html is not present in iOS app.
Comment 3 How can I remove my account? 2019-03-30 18:32:23 UTC
Probably we should not show such a menu entry in a mobile app. Perhaps not when using server-based Online from a mobile device browser either. "Shortcut" surely is a desktop concept.
Comment 4 How can I remove my account? 2019-04-05 16:02:45 UTC
Although I notice that iOS itself and some well-known apps do support keyboard shortcuts, see https://www.laptopmag.com/articles/every-ipad-keyboard-shortcut for instance. So maybe we should make the dialog show up?

On the other hand, I wonder if our shortcuts on iOS are for PC-style keyboard, or an Apple style, with the ⌘ key mostly used instead of the Control key for instance? (As can be deduced from the link above, the expectation presumably is that shortcuts should be Mac-style, not Windows/Linux -style.)
Comment 5 How can I remove my account? 2019-04-05 16:04:40 UTC
Created attachment 150549 [details]
Tentative patch to hide them

But I will check how hard it would be to make the dialog showing the shortcuts appear instead. (And check whether they actually work...)
Comment 6 How can I remove my account? 2019-04-05 18:03:57 UTC
Nah, the keyboard shortcut window is a HTML file that the code wants to display using HTTP, so would require different code in the app case. Will just remove the menu entry for now.
Comment 7 Commit Notification 2019-04-05 18:16:39 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/online/+/e8ba2ca35a79bb3b721f99ba662dee6f40d9aec1%5E%21

tdf#124177: Don't show the Help > Keyboard shortcuts menu entry in mobile apps