543317 - Remove unused toolbarmode and buttonstyle attributes

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Dão Gottwald [:dao]]

Attachment: patch

Comment 1

[[e-mail]]

Comment on attachment 424473 [details] [diff] [review]
patch

>-toolbarbutton:hover:active[buttonstyle="text"],
>-toolbarbutton[open="true"][buttonstyle="text"] {
Confusingly buttonstyle has become mode; toolbarmode became iconsize. Unfortunately this is in pinstripe so I can't tell whether it is correct to delete this or whether it should have switched to using mode instead. Given that the rule was never used, it seems unlikely, but I'd appreciate it if you could get a second opinion. r=me for toolbarbutton.xml either way.

Comment 2

[Ryan Flint [:rflint] (ping via IRC for reviews)]

Comment on attachment 424473 [details] [diff] [review]
patch

Anyone seeking blue text in pinstripe is probably already doing the styling themselves. r=me.

Comment 3

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/ca5c79a77bd0