Closed (duplicate)
Project:
Vote Up/Down
Version:
6.x-3.x-dev
Component:
vud_node
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Oct 2010 at 19:47 UTC
Updated:
6 Nov 2010 at 10:22 UTC
Jump to comment: Most recent file
Comments
Comment #1
droplet commentedseems fine to me.
Comment #2
droplet commentedI notice what you say when I get a clean install.
Comment #3
AntiNSA commentedThe problem is now the text is gone and there are now two down red arrows.... you can see what I mean @ thecenterofthenet.com
Comment #4
AntiNSA commentedComment #5
droplet commentedmissing a end tag at your site.
check the see if any error there:
widgets\upanddown\widget.tpl.php
Comment #6
AntiNSA commentedYour right, thanks for the great module!
Comment #7
marvil07 commentedThe patch is adding an empty span with a title attribute. I suppose the problem is that there is no a tool-tip on the widget, but I really can not see any difference(I can see the tool-tip), since there is already a tool-tip in the anchor that is one level on top of the span introduced, so, can someone please explain me what I am missing?
Powered by Dreditor.
Comment #8
droplet commented@marvil07,
you missing the second changes.
we haven't hide it in CSS
Im look back to the code and think we missing patch it in #815984: Make widgets readable by screen readers
Comment #9
marvil07 commentedI think I am not getting it right the problem, the mentioned issue #815984: Make widgets readable by screen readers adds an invisible css class to help srceen readers, but tooltip is still there.
An explanation, screenshot or video would be great to understand it :-)
BTW is this happening on all widgets or some of them?
Comment #10
droplet commentedonly this widgets/upanddown/
you can see the old code:
which has NOT define any css (.element-invisible) style in stylesheet to hide the texts
http://drupalcode.org/viewvc/drupal/contributions/modules/vote_up_down/w...
"Vote down!" should be invisible text there but now is showing. (not related to tooltips )
Comment #11
droplet commentednew patch submitted to #815984: Make widgets readable by screen readers and it is duplicated i think. reopen again if it is not. thanks.