1753760 - Eliminate preprocessor %includes from sidebar stylesheets

Status: RESOLVED FIXED

(Firefox :: Theme, task)

Description

[Sam Foster [:sfoster] (he/him)]

Replace the %includes with either @import or by inlining the included stylesheets. See bug 1659584 for details. This bug should cover:

• browser/themes/{platform}/syncedtabs/sidebar.css
  • browser/themes/shared/syncedtabs/sidebar.inc.css
• browser/themes/{platform}/places/sidebar.css
  • browser/themes/shared/places/sidebar.inc.css

Comment 1

[Bernard Igiri]

Attachment: Bug 1753760 - Remove preprocessor includes from sidebar r=Dao!

• Replaced preprocessor includes with CSS imports for places/sidebar
• Replaced preprocessor includes with media queries for syncedtabs/sidebar since the platform specific CSS files were small.

Comment 2

[Bernard Igiri]

Attachment: Bug 1753760 - Rename inc.css files to .css r=Dao!

Renamed .inc.css files to .css so that the manifest doesn't alter file names.

Depends on D139990

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8394e008936b
Remove preprocessor includes from sidebar r=dao
https://hg.mozilla.org/integration/autoland/rev/6630c5fd5a15
Rename inc.css files to .css r=dao

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/8394e008936b
https://hg.mozilla.org/mozilla-central/rev/6630c5fd5a15