Skip to content

Do not hide title links on hidden comments on mobile#416

Open
johanfleury wants to merge 1 commit intolinuxfrorg:mainfrom
johanfleury:fix/keep-hidden-comment-link-on-mobile
Open

Do not hide title links on hidden comments on mobile#416
johanfleury wants to merge 1 commit intolinuxfrorg:mainfrom
johanfleury:fix/keep-hidden-comment-link-on-mobile

Conversation

@johanfleury
Copy link

When browsing DLFP on mobile, title links to hidden comments are disabled which is not the case on desktop.

Those are the only two places disabling pointer-events:

$ git grep pointer-events | wc -l
0

@Oumph
Copy link
Member

Oumph commented Mar 19, 2026

Not specific to "hidden" comments btw?

Initial ticket: https://linuxfr.org/suivi/amelioration-navigation-en-mobilite
(mentioning the workaround with '#')

@johanfleury
Copy link
Author

Indeed, this behavior is not limited to hidden comments, but it's specifically an issue for those given you can't read them at all if you're not signed in (which is a big difference between desktop and mobile).

Initial ticket: https://linuxfr.org/suivi/amelioration-navigation-en-mobilite

Ah, I couldn't find it.

(mentioning the workaround with '#')

This is not really a workaround, it's just an anchor, it doesn't open the comment.

@Oumph
Copy link
Member

Oumph commented Mar 22, 2026

Trying to sum up:

  • without authentication:
    • "desktop" version: comment title is [^] to get to previous in thread comment, '#' to get to the comment in the current page, and link on title to see the dedicated page for that comment ("/nodes/XXX/comments/XXX" path). If comment is hidden, # won't display the hidden comment, but link on title will.
    • "mobile" version: same thing for [^] and '#'. No link on title, there is no way to display an hidden comment / to access the dedicate page for the comment or to expand the comment
  • with authentication: you have [-] / [+] to expand/unexpand an hidden comment

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