Opened 16 years ago
Closed 16 years ago
#740 closed Defect (fixed)
Some forum links have no separation between them
Reported by: | MichaelRoberts | Owned by: | davea |
---|---|---|---|
Priority: | Undetermined | Milestone: | Undetermined |
Component: | Web - Forums | Version: | |
Keywords: | Cc: |
Description
The correction for ticket #309 also updated some of the forum links so that the text of each link appears with no intervening inactive text.
Apart from being fairly illegible this violates W3C accessibility guidelines:
Checkpoint 10.5: Until user agents (including assistive technologies) render adjacent links distinctly, include non-link, printable characters (surrounded by spaces) between adjacent links.
If using text links rather than buttons, you can put ' | ' (or whatever) between each link, for example.
Change History (9)
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
It's as currently seen on seti@home - see this thread for example.
comment:3 Changed 16 years ago by
The white.css file on seti does not include the changes from changeset 16047. Only the code was deployed there - not the new layout for the buttons.
comment:4 Changed 16 years ago by
Thanks for the quick explanation. I'll post it to that thread and let's hope the seti@home forum configuration can be sorted out 'soon'!
comment:5 follow-ups: 6 8 Changed 16 years ago by
Links should have space in HTML anyway. Adding margin with CSS is not enough.
An online translator may attempt to translate threadSubscribe as a single word.
I tried to open the page using lynx to show how the links are unreadable there, but looks like the forum code blocks lynx user agent. What the hell?
comment:6 follow-up: 7 Changed 16 years ago by
Replying to Nicolas:
Links should have space in HTML anyway. Adding margin with CSS is not enough.
Yes, please keep this ticket open. This will be addressed in an upcoming commit (going through testing during the weekend).
I tried to open the page using lynx to show how the links are unreadable there, but looks like the forum code blocks lynx user agent. What the hell?
No this is someone who has added a side-effect in get_logged_in_user() - probably to avoid bots. I'm not sure that's a good way of doing it...
comment:7 Changed 16 years ago by
Forgot to mention that SETI@home has now updated white.css so that the 99.9% of the users who are using regular browsers should no longer be bothered.
comment:8 Changed 16 years ago by
Replying to Nicolas:
Links should have space in HTML anyway. Adding margin with CSS is not enough.
Fixed in revision 16077 and 16076.
Rather than using a hardcoded visual seperator (like "|" suggested in the ticket) we will describe what the data is (a list of actions in the form of links) and then format the visual appearance using CSS. This has the added benefit that you may style them as buttons (current default in white.css), buttons with icons on them, a vertical list of links, a menu or even a dropdown box etc. etc. without having to change the HTML code. And it should now correctly display as a list of links in lynx/links/mobile phones/screenreaders/translators too.
comment:9 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Did you deploy the changes to white.css on your project? Where did you encounter this issue?