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

Investigate: Let users edit track name in place

    Details

    • Type: New Feature
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:

      Description

      A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.

        Attachments

          Activity

          mason Mason Meyer (Inactive) created issue -
          mason Mason Meyer (Inactive) made changes -
          Field Original Value New Value
          Rank Ranked higher
          mason Mason Meyer (Inactive) made changes -
          Epic Link IGBF-440 [ 15498 ]
          mason Mason Meyer (Inactive) made changes -
          Rank Ranked lower
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Epic Link IGBF-440 [ 15498 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Story Points 2
          Labels Intermediate
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Epic Link IGBF-1765 [ 17855 ]
          ann.loraine Ann Loraine made changes -
          Sprint Summer 2019 Sprint 11 [ 70 ]
          ann.loraine Ann Loraine made changes -
          Issue Type Improvement [ 4 ] New Feature [ 2 ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Investigate: Can we allow users to double-click the track name to activate editing mode and then type in a new name there? (similar to how this works when renaming a file using a file browser GUI on Windows or Mac?)

          Show
          ann.loraine Ann Loraine added a comment - - edited Investigate: Can we allow users to double-click the track name to activate editing mode and then type in a new name there? (similar to how this works when renaming a file using a file browser GUI on Windows or Mac?)
          Hide
          ann.loraine Ann Loraine added a comment -

          In addition, can we add a "settings" icon to track labels to activate the context (right-click) menu?

          Show
          ann.loraine Ann Loraine added a comment - In addition, can we add a "settings" icon to track labels to activate the context (right-click) menu?
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          noor91zahara Noor Zahara (Inactive) made changes -
          Assignee Noor Zahara [ noor91zahara ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Sprint Summer 2019 Sprint 11 [ 70 ] Summer 2019 Sprint 10 [ 69 ]
          noor91zahara Noor Zahara (Inactive) made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment -

          As per discussion with Ann, placing the rename option in the context menu adds a lot of steps for the user to go through just to rename the track. And also the right click menu/ context menu needs to be re-designed, so it is a good idea not to add new things in the menu.

          A better solution would be to add a functionality wherein the user can click on the track name and it turns into an editable text box.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - As per discussion with Ann, placing the rename option in the context menu adds a lot of steps for the user to go through just to rename the track. And also the right click menu/ context menu needs to be re-designed, so it is a good idea not to add new things in the menu. A better solution would be to add a functionality wherein the user can click on the track name and it turns into an editable text box.
          noor91zahara Noor Zahara (Inactive) made changes -
          Sprint Summer 2019 Sprint 10 [ 69 ] Summer 2019 Sprint 11 [ 70 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Thanks Noor!

          Moving this out of the current sprint and into the backlog for consideration later.

          Show
          ann.loraine Ann Loraine added a comment - Thanks Noor! Moving this out of the current sprint and into the backlog for consideration later.
          ann.loraine Ann Loraine made changes -
          Story Points 2 4
          Sprint Summer 2019 Sprint 11 [ 70 ]
          Labels Intermediate Advanced
          Assignee Noor Zahara [ noor91zahara ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          ann.loraine Ann Loraine made changes -
          Workflow Loraine Lab Workflow [ 15720 ] Fall 2019 Workflow Update [ 18749 ]
          ann.loraine Ann Loraine made changes -
          Workflow Fall 2019 Workflow Update [ 18749 ] Revised Fall 2019 Workflow Update [ 20871 ]
          Status Open [ 1 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Summary Add "Rename" option to Track Context Menu Let users edit track name in place
          ann.loraine Ann Loraine made changes -
          Description A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track.

          This would fit with the often-used convention of right-clicking a file or object to rename it found in many other operating systems and programs.

          A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.

          This would fit with the often-used convention of right-clicking a file or object to rename it found in many other operating systems and programs.

          ann.loraine Ann Loraine made changes -
          Description A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.

          This would fit with the often-used convention of right-clicking a file or object to rename it found in many other operating systems and programs.

          A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.

          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Description A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.

          A nice addition to the Track Context Menu (the right-click menu for tracks) would be to add the option to "Rename" the track or let the user click on the name (in the track) to edit it.
          ann.loraine Ann Loraine made changes -
          Summary Let users edit track name in place Investigate: Let users edit track name in place
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 [ 181 ]
          jsirigin Jaya Sravani Sirigineedi made changes -
          Assignee Jaya Sravani Sirigineedi [ jsirigin ]
          jsirigin Jaya Sravani Sirigineedi made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Notes from scrum:

          • TierLabelManager java class has mouselisteners that intercept mouse interactions on the component.
          • We proposed a new interaction: a user double-clicks the track label text. The track label text then changes its appearance to look "editable." The user then types some new text and presses ENTER. The text of the label then becomes what the user entered.
          • We think we should investigate whether this is possible and make a possible plan for how this could be done. Then close the ticket.
          Show
          ann.loraine Ann Loraine added a comment - - edited Notes from scrum: TierLabelManager java class has mouselisteners that intercept mouse interactions on the component. We proposed a new interaction: a user double-clicks the track label text. The track label text then changes its appearance to look "editable." The user then types some new text and presses ENTER. The text of the label then becomes what the user entered. We think we should investigate whether this is possible and make a possible plan for how this could be done. Then close the ticket.
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment -

          The Track is completely designed using Graphics2D, the label is shown by the drawString method, which makes it difficult to change the label to editable. We might have to create a custom component. It might be easier to add the rename in the context menu in terms of effort.

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - The Track is completely designed using Graphics2D, the label is shown by the drawString method, which makes it difficult to change the label to editable. We might have to create a custom component. It might be easier to add the rename in the context menu in terms of effort.
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment -

          Investigating a way to perform any action when the text on the TierLabel is clicked. Right now, looking into the classes TierLabelManager which is responsible for the mouse interactions, it has all the listeners related to the track label, TierLabelGlyph which is responsible for the graphics design of the entire label, and the class TrackPreferencesPanel to see how the existing rename is implemented.

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - Investigating a way to perform any action when the text on the TierLabel is clicked. Right now, looking into the classes TierLabelManager which is responsible for the mouse interactions, it has all the listeners related to the track label, TierLabelGlyph which is responsible for the graphics design of the entire label, and the class TrackPreferencesPanel to see how the existing rename is implemented.
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 [ 181 ] Fall 5, Fall 6 [ 181, 182 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment -

          Written some code in TierLabelManager for the mouseClick event to just rename the label at the moment, it isn't working as expected so debugging the code to see the issue.

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - Written some code in TierLabelManager for the mouseClick event to just rename the label at the moment, it isn't working as expected so debugging the code to see the issue.
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment - - edited

          Made changes to the code to rename the Label to a hardcoded string, which is now working but it is getting renamed if the user clicks anywhere in the rectangular area of the label need to change it to work only when the user clicks on the string. Also, have to add a condition to check the Coordinates tier as well, as we are not allowing users to rename it. And one more thing is that it's not getting renamed when the user drags it, which is a good thing
          Before clicking:

          After clicking:

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - - edited Made changes to the code to rename the Label to a hardcoded string, which is now working but it is getting renamed if the user clicks anywhere in the rectangular area of the label need to change it to work only when the user clicks on the string. Also, have to add a condition to check the Coordinates tier as well, as we are not allowing users to rename it. And one more thing is that it's not getting renamed when the user drags it, which is a good thing Before clicking: After clicking:
          jsirigin Jaya Sravani Sirigineedi made changes -
          Attachment Screenshot 2023-11-13 at 9.39.51 AM.png [ 18097 ]
          Attachment Screenshot 2023-11-13 at 9.39.37 AM.png [ 18098 ]
          jsirigin Jaya Sravani Sirigineedi made changes -
          Attachment Screenshot 2023-11-13 at 9.39.51 AM.png [ 18097 ]
          jsirigin Jaya Sravani Sirigineedi made changes -
          jsirigin Jaya Sravani Sirigineedi made changes -
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment - - edited

          Code changes are done to rename the label when the user double click on the middle of the track label (where the name appears) and a popup JOptionPane is shown in order for the user to enter the new name (see below screenshot), once the user enters the new name and clicks ok it will rename the label, if he wishes to not rename he can click cancel or close the window.
          Branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-457

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - - edited Code changes are done to rename the label when the user double click on the middle of the track label (where the name appears) and a popup JOptionPane is shown in order for the user to enter the new name (see below screenshot), once the user enters the new name and clicks ok it will rename the label, if he wishes to not rename he can click cancel or close the window. Branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-457
          Hide
          jsirigin Jaya Sravani Sirigineedi added a comment -

          As explained in the above comments, the Track Label is completely designed using graphics, we will be able to track mouse events but introducing JLabel or any tet box into the graphics is kind of hard, so currently this is implemented using a popup dialog box when the user double clicks the middle part of the track label (where the name appears). A pull request has been raised to a new branch in the team repository to store code for the current implementation. https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/953

          Show
          jsirigin Jaya Sravani Sirigineedi added a comment - As explained in the above comments, the Track Label is completely designed using graphics, we will be able to track mouse events but introducing JLabel or any tet box into the graphics is kind of hard, so currently this is implemented using a popup dialog box when the user double clicks the middle part of the track label (where the name appears). A pull request has been raised to a new branch in the team repository to store code for the current implementation. https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/953
          jsirigin Jaya Sravani Sirigineedi made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          jsirigin Jaya Sravani Sirigineedi made changes -
          Assignee Jaya Sravani Sirigineedi [ jsirigin ]
          nfreese Nowlan Freese made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ]
          Hide
          nfreese Nowlan Freese added a comment - - edited

          Testing on Mac
          Built Sravani's IGBF-457 IGB branch successfully.
          The renaming feature works as described.

          Ready for pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/953

          Show
          nfreese Nowlan Freese added a comment - - edited Testing on Mac Built Sravani's IGBF-457 IGB branch successfully. The renaming feature works as described. Ready for pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/953
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Ann Loraine [ aloraine ]
          nfreese Nowlan Freese made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          nfreese Nowlan Freese made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Merged the branch to the team repository. Branch installers ready for testing.

          Show
          ann.loraine Ann Loraine added a comment - - edited Merged the branch to the team repository. Branch installers ready for testing.
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          nfreese Nowlan Freese made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          nfreese Nowlan Freese made changes -
          Assignee Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          To test:

          • Download, install, and run branch installer from team repository; name should contain text "IGBF-457"

          Other suggestion for testing:

          • If not already done, check how the new interface component looks in Linux, Windows, Mac
          Show
          ann.loraine Ann Loraine added a comment - - edited To test: Download, install, and run branch installer from team repository; name should contain text " IGBF-457 " Other suggestion for testing: If not already done, check how the new interface component looks in Linux, Windows, Mac
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5, Fall 6 [ 181, 182 ] Fall 5, Fall 6, Fall 7 [ 181, 182, 183 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          nfreese Nowlan Freese added a comment - - edited

          Testing on Mac

          Downloaded, installed, and ran branch installer for IGBF-457. Everything worked correctly.

          The appearance of the new interface component in Mac was the same as other/previous IGB interface components, i.e. it looked normal.

          To test:
          Download and install the IGBF-457 branch installer.
          Start IGB.
          Select the human genome.
          Double-click on the RefSeq Curated track name.
          Check that a window appears near where you double-clicked.
          Check that the window allows you to modify the track's name.

          Show
          nfreese Nowlan Freese added a comment - - edited Testing on Mac Downloaded, installed, and ran branch installer for IGBF-457 . Everything worked correctly. The appearance of the new interface component in Mac was the same as other/previous IGB interface components, i.e. it looked normal. To test: Download and install the IGBF-457 branch installer. Start IGB. Select the human genome. Double-click on the RefSeq Curated track name. Check that a window appears near where you double-clicked. Check that the window allows you to modify the track's name.
          kgopu Kaushik Gopu made changes -
          Assignee Kaushik Gopu [ kgopu ]
          Hide
          kgopu Kaushik Gopu added a comment - - edited

          Tested on Windows.

          • Fetched the IGBF-457 branch and ran it on a Windows machine.
          • Followed the test instructions provided in the notes.
          • The renaming feature works as described
          Show
          kgopu Kaushik Gopu added a comment - - edited Tested on Windows. Fetched the IGBF-457 branch and ran it on a Windows machine. Followed the test instructions provided in the notes. The renaming feature works as described
          kgopu Kaushik Gopu made changes -
          Assignee Kaushik Gopu [ kgopu ]
          Hide
          nfreese Nowlan Freese added a comment -

          Testing complete, closing ticket.

          Show
          nfreese Nowlan Freese added a comment - Testing complete, closing ticket.
          nfreese Nowlan Freese made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          nfreese Nowlan Freese made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          nfreese Nowlan Freese made changes -
          Assignee Jaya Sravani Sirigineedi [ jsirigin ]

            People

            • Assignee:
              jsirigin Jaya Sravani Sirigineedi
              Reporter:
              mason Mason Meyer (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: