OS specific css but common icons?

[moonbat]

I've had complaints about the background color of the toolbar button not displaying properly on the default theme on Windows, which is not verifiable by me since I only have Linux here.
I thought of having separate CSS files for windows and linux by changing the chrome manifest, and putting the icon files in a common subdirectory 'all' - but it doesn't work and the icon is garbled.
This is my chrome.manifest -

Code: Select all

content	pmplayer											content/
skin	pmplayer							 classic/1.0	skin/all
skin	pmplayer-os							 classic/1.0	skin/mac	os=Darwin
skin	pmplayer-os							 classic/1.0	skin/unix	os=Linux
skin	pmplayer-os							 classic/1.0	skin/win	os=WINNT
locale	pmplayer							 en-US			locale/en-US/
overlay chrome://navigator/content/navigator.xul			chrome://pmplayer/content/browserOverlay.xul application={a3210b97-8e8a-4737-9aa0-aa0e607640b9}
overlay chrome://browser/content/browser.xul 				chrome://pmplayer/content/browserOverlay.xul application={8de7fcbb-c55c-4fbe-bfc5-fc555c87dbc4}
overlay chrome://browser/content/browser.xul 				chrome://pmplayer/content/browserOverlay.xul application={ec8030f7-c20a-464f-9b0e-13a3a9e97384}
style 	chrome://global/content/customizeToolbar.xul 		    chrome://pmplayer-os/skin/overlay.css

So from the above, I've put the separate CSS in sudirectories win,mac and unix, and the extension icons are in skin/all. The CSS refers to the icons using a chrome://pmplayer/skin/ prefix, and my XUL files refer to the CSS as chrome://pmplayer-os/skin/overlay.css

This doesn't work and the icon in the toolbar shows all the standard icons mashed together (as it does when it can't find the icons).
Is this(have separate CSS for each OS but have icon files in common under 'all' because they're the same) possible to do, and if so how does one go about it?

[moonbat]

Fixed it - I didn't have trailing / for the paths under skin/

[Lootyhoof]

You might also want to remove the os=Linux from the unix path so that it works for other GTK-based systems too (e.g. Solaris). It will be overridden if on Windows/Mac anyway.

[moonbat]

Thanks, I'll do that. Barely an hour since I published the update