Closed Bug 844412 Opened 9 years ago Closed 3 years ago

Create a theme directory for shared /toolkit theme resources

Categories

(Toolkit :: Themes, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: Dolske, Assigned: Dolske)

Details

Attachments

(1 file)

Bug 838244 seems like a pretty swell idea. Let's do that in /toolkit too.
(This just moves the common in-content stuff for video controls and plugins, I'm sure there's more...)
Comment on attachment 717460 [details] [diff] [review]
Patch v.0 (untested)

>--- a/toolkit/themes/winstripe/mozapps/plugins/pluginProblem.css
>+++ b/toolkit/themes/shared/plugins/pluginProblem.css
>@@ -180,16 +180,25 @@ html|a {
> .closeIcon:hover {
>   background-position: -16px 0;
> }
> 
> .closeIcon:hover:active {
>   background-position: -32px 0;
> }
> 
>+%ifdef XP_MACOSX
>+@media (min-resolution: 2dppx) {
>+  .closeIcon {
>+    background-image: url("chrome://global/skin/icons/close@2x.png");
>+    background-size: auto 16px;
>+  }
>+}
>+%endif

As a basic principle, I'd prefer that instead of introducing ifdefs in shared/ we kept pinstripe/mozapps/plugins/pluginProblem.css, made it %include the shared pluginProblem.css and appended the @media (min-resolution: 2dppx) block. And if the differences add up, just keep the three files.
(In reply to Dão Gottwald [:dao] from comment #2)
> As a basic principle, I'd prefer that instead of introducing ifdefs in
> shared/ we kept pinstripe/mozapps/plugins/pluginProblem.css, made it
> %include the shared pluginProblem.css and appended the @media
> (min-resolution: 2dppx) block. And if the differences add up, just keep the
> three files.

Yeah, I agree.
It looks to me like this happened in bug 888510... can we close this one?
videocontrols.css (and probably others) hasn't been moved to the shared directory, but I think we can close this one and open a new one for the files that still need to be moved.
Can we close this ?
Flags: needinfo?(dao+bmo)
Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(dao+bmo)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.