Copied from bug 303898, see testcase - https://bugzilla.mozilla.org/attachment.cgi?id=264095 A double separator appears in the context menu when an image from a *sub-frame* is highlighted. Steps to reproduce: 1. Visit testcase. 2. Highlight image, be careful not to select any text, only the image. 3. Activate the context menu over the highlighted image. Result: A double separator in the menu.
Attachment #531144 - Flags: review?(neil)
Comment on attachment 531144 [details] [diff] [review] Port Firefox patch v1.0 I don't think this is right. The "quick" fix is to change the test for context-sep-selectall to only show the separator if actual text is selected. The "better" fix is to reorder the items. I don't think that "This Frame" belongs between "Search ... for ..." and "View Selection Source".
Attachment #531144 - Flags: review?(neil) → review-
This patch: * Moves frame menu to just above spell-separator
Comment on attachment 531281 [details] [diff] [review] Move frame menu instead patch v2.0 [Checked in: Comment 3] http://hg.mozilla.org/comm-central/rev/5484b8035db5
Attachment #531281 - Attachment description: Move frame menu instead patch v2.0 → Move frame menu instead patch v2.0 [Checked in: Comment 3]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.2a1
You need to log in before you can comment on or make changes to this bug.