Skip to content

[Fix] Fixing appearance of noscript warning#13

Open
nikita-kun wants to merge 1 commit intogwern:masterfrom
nikita-kun:patch-1
Open

[Fix] Fixing appearance of noscript warning#13
nikita-kun wants to merge 1 commit intogwern:masterfrom
nikita-kun:patch-1

Conversation

@nikita-kun
Copy link
Copy Markdown

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.
@gwern
Copy link
Copy Markdown
Owner

gwern commented Jan 24, 2025

We agree that the existing noscript is not very good, but Achmiz wants to take a different approach and is still working on the refactoring.

@gwern
Copy link
Copy Markdown
Owner

gwern commented Apr 1, 2026

Historical note: the noscript was tweaked in 66bd82f in preparation for a fix, but it never got finished, so right now in Firefox the JS-blocked appearance is broken.
xwd-17750108632393443

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.

2 participants