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

Fix deprecation warning in SCSS compilation #2145

Merged
merged 4 commits into from
Jun 28, 2023
Merged

Conversation

mortenpi
Copy link
Member

If I understand correctly, it is because we wrap a lot of the SCSS variables in html.theme--#{$themename} { in the dark theme.

Close #1766, close #1983.

@mortenpi mortenpi added Format: HTML Related to the default HTML output Type: Bugfix labels Jun 22, 2023
@mortenpi mortenpi enabled auto-merge (squash) June 28, 2023 10:21
@mortenpi mortenpi merged commit 7758bd8 into master Jun 28, 2023
@mortenpi mortenpi deleted the mp/fix-sass-warning branch June 28, 2023 10:27
@juliohm
Copy link
Contributor

juliohm commented Jun 28, 2023

It would be nice to have a patch release for this :)

@juliohm
Copy link
Contributor

juliohm commented Jun 30, 2023

ping @mortenpi

mortenpi added a commit that referenced this pull request Jul 3, 2023
* Fix deprecation warning in SCSS compilation

(cherry picked from commit 7758bd8)
@mortenpi mortenpi mentioned this pull request Jul 3, 2023
@mortenpi
Copy link
Member Author

mortenpi commented Jul 3, 2023

Sure, why not. It doesn't seem to conflict with any of the other changes on master: #2162.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Format: HTML Related to the default HTML output Type: Bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warnings with custom theme files Deprecation warning from Documenter.jl source code
2 participants