Closed Bug 1659583 Opened 2 years ago Closed 5 months ago

[meta] Remove preprocessor conditionals from CSS files

Categories

(Firefox :: General, task)

task

Tracking

()

RESOLVED FIXED
Tracking Status
firefox100 --- fixed

People

(Reporter: Gijs, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

For more contexts, see bug 1659444.

Query that currently captures all of these: https://searchfox.org/mozilla-central/search?q=%25if&path=css .

There are broadly the following cases:

  1. %if 0, especially for license headers. These can just be converted to normal comments. This is now covered by bug 1747459.
  2. CSS that can move from "shared" files to OS-specific files
  3. CSS using MENUBAR_CAN_AUTOHIDE, which can probably switch to using an attribute on the root (moving the ifdef into browser.xhtml, where it's already used anyway)
  4. CSS that is platform-specific and can switch to using an attribute, media query, or other mechanism
  5. CSS that is inside the UA stylesheets. There we can use something like:
/*
%ifdef MOZ_WIDGET_GTK
*/
...
/*
%endif
*/

to hide the preprocessor markup from a linter.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: General → WebRTC
Product: Firefox → Core
Component: WebRTC → General
Product: Core → Firefox
See Also: → 1747459
Depends on: 1747459
See Also: 1747459
Depends on: 1699068
Depends on: 1752830
Depends on: 1752832
Depends on: 1752833
Depends on: 1752837
Keywords: meta
Summary: Remove preprocessor conditionals from CSS files → [meta] Remove preprocessor conditionals from CSS files
Depends on: 1755670
Depends on: 1755688
Depends on: 1756655
Depends on: 1756656
Depends on: 1756676
Depends on: 1757639
Depends on: 1757642
Depends on: 1758944

This has no more dependencies and there are no more %ifs in our CSS files

Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Depends on: 1762913
You need to log in before you can comment on or make changes to this bug.