Bug 122572 - Dialog theming/UX - edit menu / text handles
Summary: Dialog theming/UX - edit menu / text handles
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: iOS Editor (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other iOS
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard: target:6.3.0 target:6.4.0
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-08 19:47 UTC by Nicolas Christener
Modified: 2019-08-30 08:44 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
screenshot of the current menu above a selection (180.64 KB, image/png)
2019-05-31 13:49 UTC, How can I remove my account?
Details
screenshot of work in progress (223.24 KB, image/png)
2019-05-31 15:29 UTC, How can I remove my account?
Details
newer screenshot (750.84 KB, image/png)
2019-06-03 07:54 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-01-08 19:47:42 UTC
Description:
The orange handles to select text ressemble (older) Android handles.

iOS (11.x) has a different style and some additional features.

* the handles are blue and have a dot which can be used to drag the handle
* above the selection a gray menu is shown with additional features

Apple describes this in their HIG document:
https://developer.apple.com/design/human-interface-guidelines/ios/controls/edit-menus/

I would expect the following actions to be available in the menu which is shown above the selection:
* cut (cut the marked text)
* copy (copy the marked text)
* delete (deletes the marked text)
* correction... (opens the spell checking dialog)
* comment... (add a comment)
* style... (opens the dialog to edit the paragraph style)

Steps to Reproduce:
1. Create a writer document in the iOS app
2. Double tap some text to trigger the text selection handles

Actual Results:
Orange text handles are shown (one at the beginning of the selection and one at the end of the selection).

Expected Results:
Blue text handles with the iOS design should be shown. Additional a menu on top of the selection should be shown where the user can access actions which make sense in the current context.


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 Aron Budea 2019-02-12 04:00:09 UTC
Let's confirm.
Comment 2 How can I remove my account? 2019-05-31 13:49:04 UTC
Created attachment 151809 [details]
screenshot of the current menu above a selection

Changing the look of the handles should be easily doable, I think.

The menu above is harder. We already have a menu with graphical buttons for Cut, Copy, and Paste (see screenshot), but currently it doesn't appear immediately when something has been selected, but you have to tap lightly once more on the selection. (Not sure if that is intentional or a bug.) Also, the menu seems to overlap the selected text. Changing the menu to look like Apple's, with text instead, is doable, but not trivial, I think. Adding more entries to this menu is far from trivial either.
Comment 3 How can I remove my account? 2019-05-31 14:00:33 UTC
Some code pointers in case I forget: The menu above the selection is actually called a "context toolbar" in loleaflet, and is created in loleaflet/src/control/Control.ContextToolbar.js in _initLayout().
Comment 4 Nicolas Christener 2019-05-31 14:16:04 UTC
IMHO the menu (context toolbar) should show right after the selection was made. This is how iOS does it and I also think user would expect it.

Making it black (like the ones in native apps) and then add cut, copy, paste would already be a nice improvement.
Comment 5 How can I remove my account? 2019-05-31 15:29:25 UTC
Created attachment 151810 [details]
screenshot of work in progress

Actually, producing something that looks quite close to the native iOS horizontal context menu was not that hard after all using CSS. It is already almost working, just the text position in the cells and the position of the whole table need to be corrected a bit.
Comment 6 Commit Notification 2019-05-31 16:56:11 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Use a context toolbar in the iOS app that looks more native
Comment 7 Commit Notification 2019-05-31 17:48:40 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Use the _UNO() function instead of duplicating what it does
Comment 8 How can I remove my account? 2019-06-03 07:54:35 UTC
Created attachment 151864 [details]
newer screenshot

Tweaked it a bit more. I see that there is a thin rectangular border around it still that shows up outside the routed corner, some further small change to the CSS is needed.
Comment 9 Commit Notification 2019-06-03 11:12:42 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Don't underline the entries in the iOS selection context menu
Comment 10 Commit Notification 2019-06-03 11:12:48 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Avoid thin rectangular border around context toolbar on iOS
Comment 11 How can I remove my account? 2019-07-30 09:52:35 UTC
More functionality will become available in the "context menu", which is a technically quite different thing than the "context bar" that has just Cut, Copy and Paste. That work is however in another branch at the moment, but will be ported to the branch from which the iOS app is built. We will try to make the context menu use the same native lookalike styling as the context bar. Note that I mistakenly used the term "context menu" for the context bar in previous comments and commits. Easy to get confused...
Comment 12 How can I remove my account? 2019-08-16 10:48:28 UTC
Seems that even after 5448916f98c1480cfbc64f58e6d4f1cdb3fd133f ("Temporarily resurrect the context toolbar, but for the iOS app only") the context toolbar did not always show up when tapping on a selection. This seems to be fixed by https://gerrit.libreoffice.org/#/c/77574/ .
Comment 13 Commit Notification 2019-08-20 08:53:18 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Temporarily resurrect the context toolbar, for the iOS app only
Comment 14 Commit Notification 2019-08-20 08:56:00 UTC
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

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

tdf#122572: Make the context toolbar show up more reliably
Comment 15 How can I remove my account? 2019-08-30 08:44:52 UTC
Should be fixed now, also in the branch from which the iOS app is built, resolving.