Prevent accidently blocking images via SeaMonkey mailcontext and context popup menu.
Categories
(SeaMonkey :: Passwords & Permissions, task)
Tracking
(seamonkey2.53+ fixed, seamonkey2.57esr? affected)
People
(Reporter: frg, Assigned: frg)
Details
(Whiteboard: SM2.53.5)
Attachments
(1 file, 1 obsolete file)
11.54 KB,
patch
|
frg
:
review+
frg
:
approval-comm-release+
frg
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
User now and then manage to block images via the popup menu. The block and unblock items are just too near to view and copy image and if you are a bit careless this happens.
I think moving them under a separater should fix this for almost all cases. Some users might still need them so we donÄt want to remove the items.
Also a comment in the mailcontext popup states wrongly that this is not used and the enable/disable code there causes the exact problems mentioned with working only on second open so update this too.
![]() |
Assignee | |
Updated•5 years ago
|
![]() |
Assignee | |
Comment 1•5 years ago
|
||
[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: support issues/calls with broken websites
Testing completed (on m-c, etc.): 2.53.5
Risk to taking this patch (and alternatives if risky): moderate so lets not take it to 2.53.4
String changes made by this patch: none
![]() |
Assignee | |
Comment 3•5 years ago
|
||
Version 1.1. Use shouldShowSeparater from nsContextMenu.js and update its comment and body (var -> let).
r/a+ from IanN retained
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/726dd53ea621
Move block image context menu items to separate entry and cleanup mail context menu handling. r=IanN
![]() |
Assignee | |
Updated•5 years ago
|
![]() |
Assignee | |
Comment 5•5 years ago
|
||
Description
•