= Description = 1. Comments in Moin1.6 are only marked by a div-wrapper causing the comment to be displayed in some nicely colored box. Alas with this solution there is no way for people using screenreaders to recognize when a comment starts and when a comment ends since the difference is only optically marked by colors. 2. But this is not only a topic for visually impaired people. Normal users are also confronted with this when css is disabled. There again is no way now to tell where a comment starts and where a comment ends. == Steps to reproduce == ## Describe the steps needed to reproduce the bug. If we can't reproduce it, we probably can't fix it. 1. do this... == Example == ##Add URL that show the bug, screenshot or test wiki markup that fail... ##URL: ##attachment:screenshot.png ##{{{ ##Example wiki markup that fail ##}}} == Component selection == ## Where you think is this bug happening ? (general, plugin [plugin name], theme [theme name], ... * general == Details == ## If you got a traceback, please save the traceback page as html and attach here: ## attachment:traceback.html ## if the bug is in this wiki, just kill the table and write: This Wiki. ## If a traceback is not available, please fill in the details here: || '''!MoinMoin Version''' || || || '''OS and Version''' || || || '''Python Version''' || || || '''Server Setup''' || || || '''Server Details''' || || || '''Language you are using the wiki in''' (set in the browser/UserPreferences) || || == Workaround == ## How to deal with the bug until it is fixed = Discussion = This could easily be solved by some extra text injected at the beginning and the end of the comment, which is not displayed on the screen but read by screenreaders and displayed when CSS is disabled. Here is a patch of parser/moin_text_wiki.py for Moin (moin-1-6-main-4cfb61592fe4): {{attachment:text_moin_wiki.diff}} To get this correctly work please add following CSS class to the theme's screen.css: {{{ /* Visually hidden information for screenreaders, i.e. is read but not displayed */ .screenreader_info { position: absolute; top: -10000px; left: -10000px; } }}} Add {{{display:none}}} for this class in projection.css and print.css??? -- OliverSiemoneit <> Please check the rest of what's unclear and submit a complete patch. Is screenreader_info some standardized way or just used by a single or few programs? -- ThomasWaldmann <> Naming of the css class is up to each programm, but the basic solution to move text out of the visible area with position absolute and large negative values for left, top is state-of-the-art and widely used on accessible sites. The new css class "screenreader_info" could also be reused for other purposes, see improved patch for MoinMoinBugs/AltAndTitleTextsDoNotDisplayCorrectlyInTwikiDrawPlugin or the accessible diff on AccessibleMoin -- OliverSiemoneit <> Here's an updated patch for Moin1.6 changeset 2041 (5932b2b3bbeb): [[attachment:text_moin_wiki2.diff]]. * Themes' screen.css needs: {{{.screenreader_info { position: absolute; top: -10000px; left: -10000px; } }}} * print.css and projection.css needs {{{ .screenreader_info { display: none } }}} * Sorry, no unified diff since I still have some trouble with hg.. -- OliverSiemoneit <> = Plan = ## This part is for Moin``Moin developers: * Priority: * Assigned to: * Status: ---- ## If you are a moin core developer, replace the category to Category* in these cases: ## Category MoinMoinNoBug - if this is not a bug. ## Category MoinMoinBugConfirmed - if you can confirm the bug on current code. ## Category MoinMoinBugFixed - after the bug is fixed in current code. CategoryMoinMoinBug