-
-
Notifications
You must be signed in to change notification settings - Fork 993
Use CSS instead of JS for floating docs banner [#1988] #2075
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
base: main
Are you sure you want to change the base?
Conversation
…e Solved - django#1739 Added verbose_name_plural = _("Local Django Communities") in the Meta class
Fixed incorrect pluralization of LocalDjangoCommunity in admin - Issu…
Issue 1764
Arpan8925 issue1764
@arpan8925 Could you squash your commits into one? |
Give me some time, exam is going on, trying this as soon as possible in next 2/3 days |
Sure! If you need some guidance to squash the commits, you can do it in at least two ways:
Let me know if you have trouble. |
7b7a4d9
to
8585eb2
Compare
for more information, see https://pre-commit.ci
8585eb2
to
2d60be7
Compare
for more information, see https://pre-commit.ci
updated @adamzap , review kindly |
Hi @marksweb Can you check if yours Change Request is resolved? |
djangoproject/scss/_style.scss
Outdated
} | ||
|
||
/* Modern browsers that support :has() */ | ||
@supports selector(:has(a, b)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem to work.
❯ make compile-scss
python -m pysassc djangoproject/scss/output.scss djangoproject/static/css/output.css --style=compressed
pysassc.py: error: Error: Invalid CSS after "@supports ": expected @supports condition (e.g. (display: flexbox)), was "selector(:has(a, b)"
on line 28:10 of djangoproject/scss/_style.scss
from line 4:1 of djangoproject/scss/output.scss
>> @supports selector(:has(a, b)) {
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok Checking
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue was happening for, actually pysassc does not support modern selector-based feature queries like: @supports selector(:has(#dev-warning)) {
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I left with 2 Options acutally
Using JavaScript-based detection only
or
Using PostCSS or Dart Sass via a JS or Node build chain instead of LibSass of pysassc .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, unfortunately I think we're stuck with the JavaScript approach for now. Do you agree @marksweb?
…oproject.com into arpan_issue_1988
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy with this now, providing everyone's happy with how it looks.
@arpan8925 I'm reviewing this as well. Thanks for the changes! |
Use CSS instead of JS for floating docs banner
Fixes #1988