After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 765145 - Flickering when opening context menu
Flickering when opening context menu
Status: RESOLVED OBSOLETE
Product: epiphany
Classification: Core
Component: Web Applications
3.20.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on: 772106
Blocks:
 
 
Reported: 2016-04-16 12:56 UTC by Andres Gomez
Modified: 2017-09-26 21:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Check edit action sensitivity in the web extension (16.62 KB, patch)
2017-02-13 04:48 UTC, Michael Catanzaro
none Details | Review
Check edit action sensitivity in the web extension (16.56 KB, patch)
2017-02-13 17:23 UTC, Michael Catanzaro
needs-work Details | Review

Description Andres Gomez 2016-04-16 12:56:57 UTC
When in Web Application mode, if using the secondary mouse button over a link to open the context menu, you can perceive, upon opening, the menu opening bigger and quickly flickering to resize to a smaller size.
Comment 1 Michael Catanzaro 2016-04-16 15:18:01 UTC
Yes, it's a WebKit API problem; the obvious way to fix it is to use sync IPC to check whether clipboard actions are available, but then a tab could hang the UI process so that's not OK.
Comment 2 Michael Catanzaro 2016-06-16 00:23:59 UTC
Maybe we can compute the menu items BEFORE showing the context menu.
Comment 3 Michael Catanzaro 2017-02-13 04:48:44 UTC
Created attachment 345599 [details] [review]
Check edit action sensitivity in the web extension

Check if the actions are sensitive in the web extension, and send that
information to the UI process along with the context menu, so that the
UI process has all the information needed to decide which items to show
at the time the context menu is constructed. This avoids nasty
flickering when the menu is shown, since we no longer have to
asynchronously remove unavailable items right after showing the menu.

Also, restrict the copy item to only be visible when there is a text
selection.

(Note: it depends on WebKit API that doesn't exist yet.)
Comment 4 Michael Catanzaro 2017-02-13 17:23:23 UTC
Created attachment 345647 [details] [review]
Check edit action sensitivity in the web extension

Check if the actions are sensitive in the web extension, and send that
information to the UI process along with the context menu, so that the
UI process has all the information needed to decide which items to show
at the time the context menu is constructed. This avoids nasty
flickering when the menu is shown, since we no longer have to
asynchronously remove unavailable items right after showing the menu.

Also, restrict the copy item to only be visible when there is a text
selection.
Comment 5 Michael Catanzaro 2017-03-11 16:26:45 UTC
Comment on attachment 345647 [details] [review]
Check edit action sensitivity in the web extension

It depends on WebKit API that we decided not to add.
Comment 6 Michael Catanzaro 2017-09-26 21:35:36 UTC
New patch in bug #772106.

Anyway, the flickering should be gone since 3.24. The patch in bug #772106 fixes things without bringing back the flickering.