Bug 661652 highlights a problem that can be commonly face by all developers using Panel API. Currently, with MacOS skins the panel background is gray so when you display a text with default color(black) it ends up being unreadable. We tried to fix that in bug 652548, but there may be various corner cases. So it may be better to add some note in documentation to highlight these color problems.

Here is such note. Feel free to move it or rephrase it completely. I'm aware of my low level in writing clear english documentation!

Alex, I thought some screenshots might help here, so I added some for Ubuntu and OS X. I don't have a Windows here, but if you had a chance to add one for Windows, I think that would be great. In case you do, I attached the background I've used for the other 2.

Comment on attachment 537699 [details] [diff] [review] Added some screenshots Review of attachment 537699 [details] [diff] [review]: ----------------------------------------------------------------- This improvement is really better than my "developer documentation" :) Except a minor word repetition I found it very clear. ::: packages/addon-kit/docs/panel.md @@ +27,4 @@ > [Working with Content Scripts](dev-guide/addon-development/web-content.html) > for more information. > > +The panel's default style is different for different operating systems. you may avoid "different" repetition by using, for example, "different for each operating systems."

Comment on attachment 537805 [details] [diff] [review] Added screenshots for Windows, OS X and Ubuntu Looks good to me!

Low-risk, useful change to have in 1.0, a=myk for commission during freeze.

Landed in https://github.com/mozilla/addon-sdk/commit/e65a5dd9088066225c8ada95975d57ddf73107dd.