Closed Bug 579272 Opened 13 years ago Closed 13 years ago

context menu background is transparent

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect)

x86
Linux
defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mnyromyr, Assigned: neil)

Details

Attachments

(1 file)

Right clicking results in a context menu with transparent background, because
<popup> is gone from toolkit now.
Compare SeaMonkey bug 578876 and bug 578440.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #459830 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 459830 [details] [diff] [review]
Port of attachment 458136 [details] [diff] [review] (ChatZilla)

Assuming you tested this (and also, wtf?) -- r=me.
Attachment #459830 - Flags: review?(gijskruitbosch+bugs) → review+
(In reply to comment #2)
> Comment on attachment 459830 [details] [diff] [review]
> Port of attachment 458136 [details] [diff] [review] (ChatZilla)
> 
> Assuming you tested this (and also, wtf?) -- r=me.

Whoa, actually, please to be fixing that whitespace before the function arguments. Thanks!
(In reply to comment #2)
> (and also, wtf?)
Pinstripe was accumulating duplicate style rules, and it was getting silly. In particular it needed rules for checkboxes in a popup but not in a menulist.

(In reply to comment #3)
> Whoa, actually, please to be fixing that whitespace before the function
> arguments. Thanks!
Sorry, all I did was to prefix the word "menu" to the word "popup". Is this a style change that needs to be incrementally applied to subsequent changes?
(In reply to comment #4)
> (In reply to comment #2)
> > (and also, wtf?)
> Pinstripe was accumulating duplicate style rules, and it was getting silly. In
> particular it needed rules for checkboxes in a popup but not in a menulist.
> 
> (In reply to comment #3)
> > Whoa, actually, please to be fixing that whitespace before the function
> > arguments. Thanks!
> Sorry, all I did was to prefix the word "menu" to the word "popup". Is this a
> style change that needs to be incrementally applied to subsequent changes?

Oh, yes. Sorry, the 'Woah' was not intended at you -  no need for you to apologize! - there is just not so much code which still has this old style (that I regularly look at), so I overlooked it initially, and was surprised. We try and fix it incrementally, indeed. Thanks!
Pushed changeset 015647fa7868 to venkman.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.