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

Highlight message targeted by anchor #58

Open
bkil opened this issue Apr 3, 2023 · 2 comments
Open

Highlight message targeted by anchor #58

bkil opened this issue Apr 3, 2023 · 2 comments

Comments

@bkil
Copy link

bkil commented Apr 3, 2023

Could we perhaps add something like this to the styles so that it is easier to see which message a permalink is referring to?

    div:target {
        background-color: #9fb;
    }

https://github.com/indieweb/chat.indieweb.org/blob/main/public/assets/styles.css

@aaronpk
Copy link
Member

aaronpk commented May 5, 2023

Is the current styling not working for you? It adds a yellow background and makes the text larger:

image

@bkil
Copy link
Author

bkil commented May 5, 2023

It doesn't where JavaScript is not enabled (i.e., in view-only mode), as the .hilite class is added dynamically via JavaScript. But yeah, the color should be the same. I also undo this locally with UserCSS to make it look right on small screens, but I haven't tested this extensively:

    @media (max-width: 992px) {
        :target:before {
            content: initial;
        }
    }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants