Uploaded image for project: 'Matterhorn Project'
  1. Matterhorn Project
  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 Tools
    • 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

        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:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: