VP-96: When keyboard used to activate Play button, tooltip remains visible but doesn't switch to "Pause"

Description

keyboard-a11y

Comments

• Alexey Novak commented 2012-08-31T14:23:57.285-0400

  A JQuery UI ticket was opened: http://bugs.jqueryui.com/ticket/8544

  Also an IM conversation attached:
  [10:19] <michelled> hi scott_gonzalez, are you around?
  [10:20] <scott_gonzalez> hey michelled
  [10:20] <michelled> I have a question for you about the jQuery ui tooltip - we've found a little bug in it and I'm wondering where we should report it
  [10:20] <michelled> I guess since it hasn't been released it it's not in Trac
  [10:21] <scott_gonzalez> Oh, I guess I should add the new plugins to Trac now that we've had beta and rc releases.
  [10:21] <scott_gonzalez> Give me one sec and I'll add them.
  [10:21] <michelled> thanks!
  [10:22] <scott_gonzalez> done
  [10:22] <michelled> great
  [10:22] <michelled> alexn: in your example, you used the rc for the jQuery UI tooltip, right?
  [10:23] <alexn> michelled: let me double check
  [10:23] <alexn> michelled: well it says 1.9m1 in a version.txt file but I tried to get the last one
  [10:23] <alexn> the latest version I mean
  [10:27] <michelled> alexn: ok, great - can you open a ticket in the jQuery Trac then?
  [10:28] <scott_gonzalez> 1.9m1 is pretty old.
  [10:28] <scott_gonzalez> alexn: You can get the rc from http://code.jquery.com/ui/1.9.0-rc.1/jquery-ui.js
  [10:29] <scott_gonzalez> and the theme: http://code.jquery.com/ui/1.9.0-rc.1/themes/base/jquery-ui.css
  [10:29] <alexn> scott_gonzalez: ok I will try rc1 to see if the issue we found still persists with the new code
  [10:34] <michelled> alexn: if you find that the bug still exists, please follow the community standards on reporting the bug: http://wiki.jqueryui.com/w/page/35263118/How-to-Report-Bugs
  [10:34] <alexn> sure michelled
  [10:36] <michelled> alexn: here's the Tooltip wiki page in case it helps: http://wiki.jqueryui.com/w/page/12138112/Tooltip
  [10:57] <alexn> michelled: so I recreated an issue using 1.9RC and pushed it to my gihub https://github.com/anvk/tooltip.example
  [11:45] <alexn> michelled: the ticket is here http://bugs.jqueryui.com/ticket/8544 and there is one answer already
  [11:53] <scott_gonzalez> alexn: What are you actually trying to do?
  [11:54] <scott_gonzalez> The example looks contrived.
  [11:55] <michelled> hey scott_gonzalez, we are building an HTML5 video player, and the specific place where we wanted to change the tooltip was for the play/pause button
  [11:56] <scott_gonzalez> Ok, let me put together a demo showing how to do that.
  [11:58] <alexn> scott_gonzalez: I added a comment with the usecase to the ticket
  [12:01] <scott_gonzalez> http://jsbin.com/uxoyan/1/edit
  [12:02] <scott_gonzalez> scroll down to the end of the <script> block
  [12:04] <scott_gonzalez> Perhaps we should track open tooltips and reinvoke the content method if it changes?
  [12:06] <scott_gonzalez> I don't think this would be too bad to add.
  [12:07] <scott_gonzalez> Actually, let me work from your example.
  [12:12] <scott_gonzalez> Hmm...yeah, there doesn't seem to be a good way to abstract this. I'll talk to the team about making this change.
  [12:26] <scott_gonzalez> alexn: We're going to make the change.

• Alexey Novak commented 2012-09-04T10:01:58.586-0400

  Fix resides in the latest trunk of JQuery UI. See above ticket for this

  Since the fix is not even a part of JQuery UI 1.9 RC the only one way to update it so far is to backport it.
  You will find that _on method is missing from the widget prototype if you copy tooltip code into the currently used Infusion. But after adding this missing _on() function the layout of the videoPlayer is broken. No idea why and what causes it.

  Even if the fix for the tooltip will get to the JQuery UI 1.9RC updating currently used JQuery UI in Infusion will require lots of testing and may cause changes/broken code in many many places.

• Jonathan Hung commented 2013-01-21T12:51:33.311-0500

  This no longer seems to be an issue. Checked with FF, Chrome, and Safari.