Skip to content

Conversation

@Tiagoquix
Copy link
Contributor

  • half-life.com looks fine with Dark Reader, but the special subpage half-life.com/*/halflife25 has some problems with Dark Reader ON, and it can be disabled specifically for that page.
    • * here is for language (/en/, /fr/ etc.).
  • l4d.com is already dark.
  • thinkwithportals.com is already dark, except for the background, but enabling Dark Reader there defeats the site purpose's of resembling the in-game levels.

@Myshor
Copy link
Collaborator

Myshor commented Aug 7, 2024

Except l4d.com I have doubts the rest should be added to dark list as they are not dark, and Dark Reader makes some cosmetic changes there.

portal2 page could be accepted as it looks strange but halflife makes some texts back to black.

I won't merge it. Let @alexanderby decides.

@Tiagoquix
Copy link
Contributor Author

Tiagoquix commented Aug 7, 2024

Half-life 25 years page has some problems with dark reader on (such as text going black and some missing background features). I believe it's ok to add an exception to that specific page, because with dark reader OFF the page looks fine IMO (not too bright).

For thinkwithportals your argument is valid and I respect it.

@Myshor
Copy link
Collaborator

Myshor commented Aug 7, 2024

I am strict about pages that are not fully dark. That's why I called Alexander here. 😉

@Tiagoquix
Copy link
Contributor Author

friendly ping @alexanderby

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