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

Add a Clear all button to Edit Tag and Color window

    Details

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

      Description

      When a user uses the SAMtags feature in Color by for one genome and then switches genomes during the same session, the values added in the previous operation is retained. However, triggering a mechanism that clears the values when genomes are changed could be tricky to implement. A Clear All or Delete All button might be a good compromise for this if there's no way to detect when a genome's been changed.

        Attachments

          Issue Links

            Activity

            Hide
            karthik Karthik Raveendran added a comment -

            Implemented a clear button. See commit

            Show
            karthik Karthik Raveendran added a comment - Implemented a clear button. See commit
            Hide
            pkulzer Paige Kulzer (Inactive) added a comment -

            Fetched Karthik's branch for testing on my Mac. The Clear all button appears to be working well, it removes all values from the table whether they were entered manually or via Import... I was able to Clear all values from the Color by table after switching genomes. No errors in the Log.

            Recommending PR!

            Show
            pkulzer Paige Kulzer (Inactive) added a comment - Fetched Karthik's branch for testing on my Mac. The Clear all button appears to be working well, it removes all values from the table whether they were entered manually or via Import... I was able to Clear all values from the Color by table after switching genomes. No errors in the Log. Recommending PR!
            Show
            karthik Karthik Raveendran added a comment - PR Submitted: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1078
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged, new installers built and deployed to Early Access section of bioviz.org.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged, new installers built and deployed to Early Access section of bioviz.org.
            Hide
            pkulzer Paige Kulzer (Inactive) added a comment -

            Tested on Mac, the Clear All button is working as expected. All values in the Color by table are removed after clicking this button. No errors in the Log.

            Closing this ticket!

            Show
            pkulzer Paige Kulzer (Inactive) added a comment - Tested on Mac, the Clear All button is working as expected. All values in the Color by table are removed after clicking this button. No errors in the Log. Closing this ticket!

              People

              • Assignee:
                karthik Karthik Raveendran
                Reporter:
                karthik Karthik Raveendran
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: