Skip to content

Conversation

@GalaxySH
Copy link
Contributor

@GalaxySH GalaxySH commented Nov 20, 2024

What

Fixes an issue with message highlighting in user talks on Wikipedia. Users receive notifications about messages on their user talk, clicking the notification brings them to the user talk where the message is highlighted.

Why

DR does not properly highlight the message. Disabling the color would prevent all message highlighting, and the default coloring on Wikipedia is a harsh white background. Recommend using one of the special DR background color variables.

Examples

Link with an example of it working: https://en.wikipedia.org/wiki/User_talk:PolyversialMind/Archive_1#c-Lajmmoore-20240523070600-Women_in_Red_June_2024
(highlighting disappears when you click elsewhere on page)

Before:
https://i.imgur.com/MlerCFV.png

After:
https://i.imgur.com/vrnyXYK.png

Fixes an issue with message highlighting in user talks on wikipedia
updated the color again to something that will work across more message types
@GalaxySH GalaxySH marked this pull request as ready for review November 20, 2024 21:32
@GalaxySH
Copy link
Contributor Author

I would welcome other suggestions for coloring or no coloring

@Myshor
Copy link
Collaborator

Myshor commented Nov 25, 2024

I tried to make the color not related to variable and I think the closest one to original is to use

.ext-discussiontools-init-targetcomment {
    background-color: rgba(192,192,255, 0.25);
}

It looks nice in Dark scheme and light scheme of Dark Reader.

Check if you confirm and if you like this then push this code to PR and I will merge.

If not then try to find different color. 😉

Copy link
Collaborator

@Myshor Myshor left a comment

Choose a reason for hiding this comment

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

The result with this code is not very nice. Check my comment.

improves the original color correction to a light purple that is easier on the eyes
@GalaxySH
Copy link
Contributor Author

GalaxySH commented Dec 19, 2024

The result with this code is not very nice. Check my comment.

after image after image

I think it looks much better now

@GalaxySH GalaxySH requested a review from Myshor December 19, 2024 00:52
@Myshor Myshor merged commit 667eaa6 into darkreader:main Dec 19, 2024
1 check passed
@GalaxySH GalaxySH deleted the patch-3 branch December 20, 2024 06:19
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