Needs review
Project:
Text Resize
Version:
7.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
8 Jul 2011 at 20:14 UTC
Updated:
29 Apr 2016 at 18:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
daggar commentedComment #2
matiskay commentedThat will be great, meanwhile you can override the theme function to accomplish that task.
Check the help in admin/help/text_resize
Comment #3
malc_b commentedI'd second this. The other way to do "fix" this is to make a custom block and copy the generated block code (i.e. use firebug to find it then copy to a custom block). That doesn't involve writing a function.
Comment #4
attheshow commentedThe -A text is already present in the page inside of the <a> tag. I don't see that screen readers would need more description than is given to the regular user.
Comment #5
malc_b commentedFYI my use is to provide an informative popup when users mouse over. I say "range -1 to +5" so users know they can make the text 1 smaller and 5 larger.
Comment #6
steveixa commentedSorry to reopen this old issue but this has come up with a client's project. It would be helpful if there was a tooltip that appeared on hover that says "Increase Text Size" and "Decrease Text Size".
For screen readers I don't think "A+" and "A-" is very meaningful (not that a blind person would need to change the size of the font, but at least it would inform them what the button does). A sighted person has the advantage of the image to the convey purpose of the button. But even then a more descriptive tooltip would be helpful.
It seem like it would be fairly straight forward to implement.
thanks
Comment #7
attheshow commentedFeel free to make this type of change on your own site but this will not be happening as part of the module. Sorry.
Comment #8
Tyler the Creator commentedHaving a screen reader say "-A" or "+A" isn't descriptive in the slightest and with testing with VoiceOver on Macs, the superscript isn't even read, so it's just saying "Link: A". The solution stated in the OP with putting the descriptive text in the link's title attribute actually doesn't work with screen readers, so the solution would be to put visibly hidden text in the link along with the original text.
Attached is a patch for the Drupal 7 version of the module that adds descriptive text to each button, as well as removes the HTML from the translate functions so that only the button text gets translated.
Comment #9
andrewmacpherson commented