Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSS "Important" Review #21

Open
leriomaggio opened this issue Nov 30, 2020 · 0 comments
Open

CSS "Important" Review #21

leriomaggio opened this issue Nov 30, 2020 · 0 comments
Labels
enhancement New feature or request FIX Something isn't working

Comments

@leriomaggio
Copy link
Owner

The main CSS style contains by far too many !important restrictions!

Some of them are justified to override main jupyter and jupyter-light directives.
However my impression is that some (most?) of them could be simply avoided or removed as not needed.

Already started removing some of those for code link colour in ul/ol > li items.
Therefore, a thorough review is needed in order to clarify what is really !important and what is, instead, leftover from workarounds and development work.

@leriomaggio leriomaggio added enhancement New feature or request FIX Something isn't working labels Nov 30, 2020
leriomaggio added a commit that referenced this issue Dec 1, 2020
This is the first step towards removing not-really-necessary `!important` CSS clauses.
(See #21)
leriomaggio added a commit that referenced this issue Dec 1, 2020
This is the first step towards removing not-really-necessary `!important` CSS clauses.
(See #21)
leriomaggio added a commit that referenced this issue Dec 1, 2020
Initial removal of useless important clauses (see #21)
leriomaggio added a commit that referenced this issue Mar 29, 2021
Editor CSS has been changed by:
- removing bold font for keywords (not supported by all mono fonts nor necessary)
- removed italic for comments as imposed by default Jupyter theme (see above)
- Removed some useless `!important` clauses (Continuation of #21)
- FIX `.highlight > pre` to correctly specify Code font size and line-height.
- FIX directive for `span.cm-string` that was using the wrong CSS var.
leriomaggio added a commit that referenced this issue Mar 29, 2021
This commit removes useless important clauses for:

- code cells (edit and display)
- md cells (edit)
- md cells headers (edit)
- md cells (display)
- links in md display cells (text-colour)
leriomaggio added a commit that referenced this issue Apr 8, 2021
This commit removes useless important clauses for:

- code cells (edit and display)
- md cells (edit)
- md cells headers (edit)
- md cells (display)
- links in md display cells (text-colour)
leriomaggio added a commit that referenced this issue Apr 8, 2021
This commit removes useless important clauses for:

- code cells (edit and display)
- md cells (edit)
- md cells headers (edit)
- md cells (display)
- links in md display cells (text-colour)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request FIX Something isn't working
Development

No branches or pull requests

1 participant