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] Fixing appearance of noscript warning #13

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

Conversation

nikita-kun
Copy link

Text color is changed from grey-on-white-background to the colors of the admonition CSS style. Fixed broken appearance of links. The floating block size and location are adjusted to avoid wasting space. Another revision is needed if the warning is to be made dismissable without the use of JS.

Before

1 before mobile noscript gwern
2 before mobile noscript gwern
1 before desktop noscript gwern
2 before desktop noscript gwern

After

1 after mobile noscript gwern
2 after mobile noscript gwern
1 after desktop noscript gwern
2 after desktop noscript gwern

Text color is changed from grey-on-white-background to the colors of the admonition CSS style. Fixed broken appearance of links. The floating block size and location is adjusted to avoid wasting space. Another revision is needed if the warning is to be made dismissable without the use of JS.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant