Skip to content
This repository has been archived by the owner on Nov 21, 2021. It is now read-only.

Add an @-moz-document block to all styles #126

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Timvde
Copy link
Owner

@Timvde Timvde commented Jan 26, 2019

This makes the styles compatible with extension ssuch as reStyle and
theoretically slightly improves performance, since styles are more
narrowly applied.

This makes the styles compatible with extension ssuch as reStyle and
theoretically slightly improves performance, since styles are more
narrowly applied.
@Timvde Timvde requested a review from Madis0 January 26, 2019 18:14
@Madis0
Copy link
Collaborator

Madis0 commented Jan 28, 2019

I don't know how official that syntax is, but let's use the comment section as suggested by the reStyle author, also let's apply the namespace too - #127

@Timvde
Copy link
Owner Author

Timvde commented Jan 30, 2019

I'm first merging in some other styles, then fix everything in one go. That's probably less work than updating all the branches at once.

The namespace should be irrelevant when using @-moz-document blocks, and I'm not sure how multiple namespaces would interfere. What's the advantage of changing the comment section?

@Madis0
Copy link
Collaborator

Madis0 commented Jan 30, 2019

What's the advantage of changing the comment section?

reStyle compatibility, it would show them in a nice manager UI.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants