Uploaded image for project: 'Opencast'
  1. Opencast
  2. MH-7355

Engage shortcuts pane: minor issues with consistency and clarity of text

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed and reviewed
    • Affects Version/s: 1.1.x (branch)
    • Fix Version/s: 1.4.0
    • Component/s: Engage User Interface
    • Labels:
      None
    • Severity:
      Cosmetic Issue
    • Steps to Reproduce::
      Hide
      Ctrl+Alt+T is "the current time for the screen reader". What does this mean? Should start with a verb to say what it will do.
      Cmd + and Cmd - : you don't need the "to". Instead use "Zoom in..." and "Minimize..." (but is this really "minimize"?)

      Steps to reproduce:
      1.
      2.
      3.
       
       Actual Results:
       
       
       Expected Results:
       
       
       Workaround (if any):
       





      Show
      Ctrl+Alt+T is "the current time for the screen reader". What does this mean? Should start with a verb to say what it will do. Cmd + and Cmd - : you don't need the "to". Instead use "Zoom in..." and "Minimize..." (but is this really "minimize"?) Steps to reproduce: 1. 2. 3.    Actual Results:      Expected Results:      Workaround (if any):  
    • License:
      Assign copyright to opencast as per ECL 2.0
    • Tags (folksonomy):

      Gliffy Diagrams

        Attachments

          Activity

          Hide
          greg_logan Greg Logan added a comment -
          Per http://opencast.3480289.n2.nabble.com/JIRA-Ticket-Cleanup-proposal-td7475080.html, this has been bulk resolved as won't fix. If this is still important to you please reopen and we can triage as appropriate.
          Show
          greg_logan Greg Logan added a comment - Per http://opencast.3480289.n2.nabble.com/JIRA-Ticket-Cleanup-proposal-td7475080.html, this has been bulk resolved as won't fix. If this is still important to you please reopen and we can triage as appropriate.
          Hide
          judy Judy Stern added a comment -
          still an issue in 1.4, albeit minor
          Show
          judy Judy Stern added a comment - still an issue in 1.4, albeit minor
          Hide
          greg_logan Greg Logan added a comment -
          What is the first shortcut supposed to do Judy? On my build it just pauses playback as far as I can tell, so it's hard for me to figure out what the text should be! The Mac shortcuts have been changed to:

          cmd + = Zoom into the player
          cmd - = Zoom out of the player

          Sound good? Nothing committed just yet, just a patch on my local machine.
          Show
          greg_logan Greg Logan added a comment - What is the first shortcut supposed to do Judy? On my build it just pauses playback as far as I can tell, so it's hard for me to figure out what the text should be! The Mac shortcuts have been changed to: cmd + = Zoom into the player cmd - = Zoom out of the player Sound good? Nothing committed just yet, just a patch on my local machine.
          Hide
          greg_logan Greg Logan added a comment -
          Over to Judy for comments.
          Show
          greg_logan Greg Logan added a comment - Over to Judy for comments.
          Hide
          judy Judy Stern added a comment -
          Those shortcuts fro zooming in and out sound good, Greg.
          Re. the first shortcut mentioned (Ctrl+Alt+T is "the current time for the screen reader") is confusing to me, too, which is why I opened the ticket. The text is missing a verb so we could know what to expect. (Doesn't seem to do anything for me, either. ) Is it supposed to Read the time, Set the focus to the time, ?. Perhaps someone from the Osnabrück team can clarify, so I'll reassign to Rüdiger.
          Show
          judy Judy Stern added a comment - Those shortcuts fro zooming in and out sound good, Greg. Re. the first shortcut mentioned (Ctrl+Alt+T is "the current time for the screen reader") is confusing to me, too, which is why I opened the ticket. The text is missing a verb so we could know what to expect. (Doesn't seem to do anything for me, either. ) Is it supposed to Read the time, Set the focus to the time, ?. Perhaps someone from the Osnabrück team can clarify, so I'll reassign to Rüdiger.
          Hide
          rrolf Ruediger Rolf added a comment -
          From my point of view it is clear what it does (making an installed screenready read the current play time to you) I have no idea how this is implemented, and I would say the user does not need to know this too.
          Show
          rrolf Ruediger Rolf added a comment - From my point of view it is clear what it does (making an installed screenready read the current play time to you) I have no idea how this is implemented, and I would say the user does not need to know this too.
          Hide
          judy Judy Stern added a comment - - edited
          Thanks, Rüdiger. Just needed that verb: the first shortcut should be " Reads the current time aloud when using a screen reader".
          Show
          judy Judy Stern added a comment - - edited Thanks, Rüdiger. Just needed that verb: the first shortcut should be " Reads the current time aloud when using a screen reader".
          Hide
          judy Judy Stern added a comment -
          Hoping you can assign the simple edit suggested to someone on your team, but if not, just unassign. (This is "Trivial" after all.)
          Show
          judy Judy Stern added a comment - Hoping you can assign the simple edit suggested to someone on your team, but if not, just unassign. (This is "Trivial" after all.)
          Hide
          greg_logan Greg Logan added a comment -
          Merged into trunk with rev 13352, 1.4.x with rev 13353.
          Show
          greg_logan Greg Logan added a comment - Merged into trunk with rev 13352, 1.4.x with rev 13353.

            People

            • Assignee:
              jamesp James Perrin
              Reporter:
              judy Judy Stern
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: