The default bug view has changed. See this FAQ.

Remove support for <popup> from all themes

RESOLVED FIXED in mozilla2.0b2

Status

()

Toolkit
Themes
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

({dev-doc-complete})

Trunk
mozilla2.0b2
dev-doc-complete
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 457044 [details] [diff] [review]
patch

We have only partial support for this right now, and it's regression prone since we don't use <popup> ourselves anymore.
Attachment #457044 - Flags: review?(enndeakin)

Comment 1

7 years ago
Comment on attachment 457044 [details] [diff] [review]
patch

This is ok but you should document this somewhere and inform others about this change. Also, Thunderbird and Seamonkey (for example) do still have <popup> throughout their code.
Attachment #457044 - Flags: review?(enndeakin) → review+
(Assignee)

Updated

7 years ago
Blocks: 578440
(Assignee)

Updated

7 years ago
Blocks: 578441
(Assignee)

Comment 2

7 years ago
updated https://developer.mozilla.org/en/Firefox_4_for_developers#Miscellaneous_XUL_changes and https://developer.mozilla.org/en/XUL/popup and filed bug 578440 and bug 578441
Keywords: dev-doc-complete
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/mozilla-central/rev/6d5be6f065c8
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3b2
This broke pop-ups from right-click context menus for addons that are in the status-bar, i.e. ABP, AniWeather, or any addon that for now works in the status-bar.  The context menu shows, but it is 'transparent'. 

Intentional that they broke, or was this for 'themes' only ?
(Assignee)

Comment 5

7 years ago
It affects add-ons that use <popup> instead of <menupopup>, it has nothing to do with the status bar per se.
Depends on: 582555

Comment 6

7 years ago
Two more extension that are affected Live HTTP Header, AdBlock Plus (bug 584566)
You need to log in before you can comment on or make changes to this bug.