Skip to content

Fixed orteil.dashnet.org homepage#15121

Draft
johnnycharlesw wants to merge 1 commit into
darkreader:mainfrom
johnnycharlesw:main
Draft

Fixed orteil.dashnet.org homepage#15121
johnnycharlesw wants to merge 1 commit into
darkreader:mainfrom
johnnycharlesw:main

Conversation

@johnnycharlesw
Copy link
Copy Markdown

I just added in a rule to fix the homepage of orteil.dashnet.org, I tested it, it does not break Cookie Clicker it seems.


================================

orteil.dashnet.org
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fixes should be sorted alphabetically


CSS
html {
background-color:#5d00a8;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leave a space after ":" and use "!important" in the end to force the values.

background-image:url(shadedEdges.png),url(bg.png);
background-size:100% 100%,auto;
}
CSS
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove repeated CSS keyword.

}
CSS
#firstBox,#Wgames,#Wtoys,#Wexps,#Wlinks
{
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The opening brace should be on the same line as the last selector (separated with space).

@Myshor
Copy link
Copy Markdown
Collaborator

Myshor commented Mar 21, 2026

I tried to apply this "fix" after adding !important to every line but it did not fix anything.

I bet someone is trying to apply background like original:
obraz

I am with big no... but as @alexanderby you started to review this then consider if we want to merge or just reject.

Now the page without any fix looks like below:
obraz

@johnnycharlesw
Copy link
Copy Markdown
Author

Well, I tried...I'll just keep it in my fork until it is ready, I guess...

@johnnycharlesw johnnycharlesw marked this pull request as draft April 6, 2026 20:34
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.

3 participants