This is a follow-up fix to bug 449832. > It changes two things: > - RTL tab margins fix > - The background and border colors of the Bookmarks toolbar are moved into a > general rule in global/toolbar.css. The border colors are tweaked a little.

Comment on attachment 339133 [details] [diff] [review] follow-up patch to fix rtl and normal toolbars +window:not([active]) toolbar, +window:not([active]) toolbarpalette { + border-top-color: rgba(255, 255, 255, 0.48); + background-color: -moz-mac-chrome-inactive; + border-bottom-color: rgba(0, 0, 0, 0.32); } This doesn't work correctly. toolbar.css is scoped to the toolbar binding and doesn't know anything about the window. So when the window's active state changes, the style rule isn't re-resolved and the toolbar has the wrong color. I'll think about what to do here.

(In reply to comment #1) > toolbar.css is scoped to the toolbar binding and > doesn't know anything about the window. So when the window's active state > changes, the style rule isn't re-resolved and the toolbar has the wrong color. You can put it in global.css. Extra points for waiting for bug 459457 so that I don't have to merge :)

We don't need to set a background color for toolbars when we have -moz-appearance: toolbar, which I'm implementing in bug 460899. Resolving as invalid.