Details

      Description

      If you right click a track label and choose clear data, the pop-up indicates that the visible data will be removed. This implies that the current view region is all that will be removed. But it clears the entire chromosome. It looks like it does not clear all the data for that track, just that chromosome.

      I think we should either modify the message so it is not misleading, or modify the method so it matches the message.

        Attachments

          Activity

          Hide
          ann.loraine Ann Loraine added a comment -

          Suggestion for new wording:

          "This will remove all data from the current sequence for the selected track(s).
          This will NOT affect other sequences not currently shown.
          Do you wish to continue?"

          Show
          ann.loraine Ann Loraine added a comment - Suggestion for new wording: "This will remove all data from the current sequence for the selected track(s). This will NOT affect other sequences not currently shown. Do you wish to continue?"
          Hide
          ssegu Sai Supreeth Segu (Inactive) added a comment -

          Irvin Naylor Please submit the link for the code change.

          Show
          ssegu Sai Supreeth Segu (Inactive) added a comment - Irvin Naylor Please submit the link for the code change.
          Show
          inaylor Irvin Naylor (Inactive) added a comment - Whoops. Here it is: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/842/igbf-1311-improve-clear-data-message
          Hide
          ann.loraine Ann Loraine added a comment -

          Please un-assign your issue when you move to a "needs review" or "needs testing" column. Otherwise, no-one will pick it up.

          Show
          ann.loraine Ann Loraine added a comment - Please un-assign your issue when you move to a "needs review" or "needs testing" column. Otherwise, no-one will pick it up.
          Hide
          nfreese Nowlan Freese added a comment -

          Tested Irvin's 1311 branch installer for Mac.

          The 2nd and 3rd sentence changes appear correctly.
          I do not see the 1st line change according to Dr. Loraine's comment.

          Irvin Naylor - please change the first sentence to "This will remove all data from the current sequence for the selected track(s)."

          Show
          nfreese Nowlan Freese added a comment - Tested Irvin's 1311 branch installer for Mac. The 2nd and 3rd sentence changes appear correctly. I do not see the 1st line change according to Dr. Loraine's comment. Irvin Naylor - please change the first sentence to "This will remove all data from the current sequence for the selected track(s)."
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          Branch link - https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-1311/

          Fixed the issue with the first line's changes not showing up properly

          Show
          inaylor Irvin Naylor (Inactive) added a comment - Branch link - https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-1311/ Fixed the issue with the first line's changes not showing up properly
          Hide
          ssegu Sai Supreeth Segu (Inactive) added a comment -

          Everything looks fine but there are 3 commits for the change d3ddc03, 8c94a43, 93439b4. Remove all commits and make the change in 1 commit.
          cc: Irvin Naylor

          Show
          ssegu Sai Supreeth Segu (Inactive) added a comment - Everything looks fine but there are 3 commits for the change d3ddc03, 8c94a43, 93439b4. Remove all commits and make the change in 1 commit. cc: Irvin Naylor
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          Fixed link - https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-1311/

          Squashed the three commits into one and fixed the commit message to reflect the change

          Show
          inaylor Irvin Naylor (Inactive) added a comment - Fixed link - https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-1311/ Squashed the three commits into one and fixed the commit message to reflect the change
          Hide
          ssegu Sai Supreeth Segu (Inactive) added a comment -

          Tested branch installer for Mac. Everything looks good to me.
          Irvin Naylor Go ahead and submit a pull request.

          Show
          ssegu Sai Supreeth Segu (Inactive) added a comment - Tested branch installer for Mac. Everything looks good to me. Irvin Naylor Go ahead and submit a pull request.
          Show
          inaylor Irvin Naylor (Inactive) added a comment - Pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/842/igbf-1311-improve-clear-data-message
          Hide
          noor91zahara Noor Zahara (Inactive) added a comment -

          Working as expected.

          Show
          noor91zahara Noor Zahara (Inactive) added a comment - Working as expected.

            People

            • Assignee:
              inaylor Irvin Naylor (Inactive)
              Reporter:
              ieclabau Ivory Blakley (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: