Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2511

Investigate and Fix: Label for an action is missing when re-binding a keystroke

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep, Fall 2: 28 Sep - 9 Oct

      Description

      In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
      If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
      The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)

      The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.

      This bug happens for the following actions:

      • Home
      • Load session
      • Import Genome [Open Genome from File]
      • Save Image
      • Thresholding
      • View Genomic Sequence
      • View Read Sequence
      • Configure Web Links

      For example:
      If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.

        Attachments

          Activity

          gchamma Jay Chamma (Inactive) created issue -
          gchamma Jay Chamma (Inactive) made changes -
          Field Original Value New Value
          Epic Link IGBF-1765 [ 17855 ]
          gchamma Jay Chamma (Inactive) made changes -
          Description In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in editing)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          Home
          Load session
          Import Genome
          Save Image
          Thresholding
          View Genomic Sequence
          View Read Sequence
          In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          Home
          Load session
          Import Genome
          Save Image
          Thresholding
          View Genomic Sequence
          View Read Sequence

          For example:
          If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.
          gchamma Jay Chamma (Inactive) made changes -
          Description In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          Home
          Load session
          Import Genome
          Save Image
          Thresholding
          View Genomic Sequence
          View Read Sequence

          For example:
          If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.
          In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          * Home
          * Load session
          * Import Genome
          * Save Image
          * Thresholding
          * View Genomic Sequence
          * View Read Sequence
          * Configure Web Links

          For example:
          If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

          I was working on this ticket today. I figured there is a discrepancy in the above actions being added to GenericActions Hashmap. Since there is no mapping present for the above actions, the value of genericAction in KeyStrokeEditPanel.java file is null and hence, the value assigned to actionDisplayName is "???" which appears in the dialog box.

          I tried to fix this by adding the action to GenericActions hashmap, but the actions which have a default keyStroke value are appearing twice in the Preferences toolbar menu. Will look into this and fix it.

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - I was working on this ticket today. I figured there is a discrepancy in the above actions being added to GenericActions Hashmap. Since there is no mapping present for the above actions, the value of genericAction in KeyStrokeEditPanel.java file is null and hence, the value assigned to actionDisplayName is "???" which appears in the dialog box. I tried to fix this by adding the action to GenericActions hashmap, but the actions which have a default keyStroke value are appearing twice in the Preferences toolbar menu. Will look into this and fix it.
          Hide
          ann.loraine Ann Loraine added a comment -

          Please explain for testers:

          • How to locate the Preferences toolbar menu in the IGB UI.
          Show
          ann.loraine Ann Loraine added a comment - Please explain for testers: How to locate the Preferences toolbar menu in the IGB UI.
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Description In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          * Home
          * Load session
          * Import Genome
          * Save Image
          * Thresholding
          * View Genomic Sequence
          * View Read Sequence
          * Configure Web Links

          For example:
          If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.
          In the Toolbar section of the Preferences panel, users can assign and re-assign keystrokes for different actions.
          If a user enters a keystroke to use for an action that is already being used for another action, the following popup window would appear letting the user know that the keystroke they're about to use for an action is already being used.
          The label for the action already assigned to the keystroke is displayed (highlighted in yellow in image)
          !https://i.imgur.com/rOgr2G5.png!

          The issue is that for some actions, if there is a keystroke assigned to them and then the user tries to re-assign the same keystroke for a different action, the same popup window would appear, but the label for the current assigned action just says "???" instead of the name of the action.
          !https://i.imgur.com/uWCH071.png!

          This bug happens for the following actions:
          * Home
          * Load session
          * Import Genome [Open Genome from File]
          * Save Image
          * Thresholding
          * View Genomic Sequence
          * View Read Sequence
          * Configure Web Links

          For example:
          If a user assigns "CTRL+J" for the action "Load session", then re-assigns "CTRL+J" to any different action, the popup window with the "???" would appear.
          ann.loraine Ann Loraine made changes -
          Sprint Summer 7: 31 Aug - 11 Sep [ 102 ] Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep [ 102, 103 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited

          The code changes are present here:
          https://bitbucket.org/skulka2710/shamika_igb/branch/IGBF-2511#diff

          To test:
          1) Open IGB
          2) Open Preferences Panel from File menu -> Preferences, or from the Shortcuts
          3) Select Toolbar section in the popup window
          4) You will see some existing keyboard shortcuts. Choose an action option and click on the respective KeyStroke cell. Press the keys you want to put as shortcut for this option. If the shortcuts already exist for other action, an error message should popup displaying the action for which the keystroke exists. Try for the actions mentioned in the description of the ticket. The proper name of the action should be displayed in the error popup instead of '???'.

          Kindly test using my branch installer

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited The code changes are present here: https://bitbucket.org/skulka2710/shamika_igb/branch/IGBF-2511#diff To test: 1) Open IGB 2) Open Preferences Panel from File menu -> Preferences, or from the Shortcuts 3) Select Toolbar section in the popup window 4) You will see some existing keyboard shortcuts. Choose an action option and click on the respective KeyStroke cell. Press the keys you want to put as shortcut for this option. If the shortcuts already exist for other action, an error message should popup displaying the action for which the keystroke exists. Try for the actions mentioned in the description of the ticket. The proper name of the action should be displayed in the error popup instead of '???'. Kindly test using my branch installer
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Summary Investigate: Label for an action is missing when re-binding a keystroke Investigate and Fix: Label for an action is missing when re-binding a keystroke
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment -

          Remove the comment from the code - //logger.info("added action: "+genericAction.getDisplay());
          Remove an extra line#113 in GenericActionHolder.java

          It works fine but one concern I observed is that in MAC when I choose command + key, IGB crashes and closes.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - Remove the comment from the code - //logger.info("added action: "+genericAction.getDisplay()); Remove an extra line#113 in GenericActionHolder.java It works fine but one concern I observed is that in MAC when I choose command + key, IGB crashes and closes.
          noor91zahara Noor Zahara (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

          Is the IGB crashing in MAC after pressing command + key occurring after my changes?
          Noor Zahara

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Is the IGB crashing in MAC after pressing command + key occurring after my changes? Noor Zahara
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment - - edited

          Looks like it is an existing issue. We can create a separate ticket to investigate that.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - - edited Looks like it is an existing issue. We can create a separate ticket to investigate that.
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

          Noor Zahara I have made the changes as suggested by you. Kindly review.
          https://bitbucket.org/skulka2710/shamika_igb/branch/IGBF-2511#diff

          Show
          shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Noor Zahara I have made the changes as suggested by you. Kindly review. https://bitbucket.org/skulka2710/shamika_igb/branch/IGBF-2511#diff
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          shamika Shamika Gajanan Kulkarni (Inactive) made changes -
          Assignee Shamika Gajanan Kulkarni [ shamika ] Noor Zahara [ noor91zahara ]
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment -

          Works fine.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - Works fine.
          noor91zahara Noor Zahara (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Assignee Noor Zahara [ noor91zahara ]
          ann.loraine Ann Loraine made changes -
          Sprint Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep [ 102, 103 ] Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep, Fall 2: 28 Sep - 9 Oct [ 102, 103, 104 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ann.loraine Ann Loraine added a comment -

          Noor Zahara - would you be able to submit the PR for Shamika since she has left for her new job?

          Show
          ann.loraine Ann Loraine added a comment - Noor Zahara - would you be able to submit the PR for Shamika since she has left for her new job?
          Show
          noor91zahara Noor Zahara (Inactive) added a comment - PR submitted - https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/831/igbf-2511-fix-for-an-action-when-re/diff
          noor91zahara Noor Zahara (Inactive) made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Merged and master branch installers built. Ready for testing.

          Show
          ann.loraine Ann Loraine added a comment - Merged and master branch installers built. Ready for testing.
          noor91zahara Noor Zahara (Inactive) made changes -
          Assignee Noor Zahara [ noor91zahara ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment -

          Working as expected.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - Working as expected.
          noor91zahara Noor Zahara (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Assignee Noor Zahara [ noor91zahara ] Shamika Gajanan Kulkarni [ shamika ]

            People

            • Assignee:
              shamika Shamika Gajanan Kulkarni (Inactive)
              Reporter:
              gchamma Jay Chamma (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: