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

Selecting multiple tracks and performing "Color by" should color all selected tracks

    Details

    • Story Points:
      1
    • Sprint:
      B - Summer 2018

      Attachments

        Issue Links

          Activity

          Hide
          mason Mason Meyer (Inactive) added a comment -

          It seems that this issue is functioning as expected. The "ColorBy" feature is applying Heatmap Settings and remembering them in a way that is intuitive to the user. After testing, I can confirm that this story is resolved because the two issues mentioned in this story have been addressed and are working as expected:

          1. If I color tracks "A" and "B" with "Heatmap Setting 1", and then color tracks "B" and "C" with "Heatmap Setting 2", track "A" also gets "Heatmap Setting 2" applied when it should remain at "Heatmap Setting 1"

          2. If I apply a heatmap setting to a track, and then select that track and another track and apply a different heatmap setting, the heatmap setting that gets remembered is not necessarily the one that is currently applied. Instead, it seems that the track's "individual heatmap setting" is always remembered, not the "group's heatmap setting"

          Since this issue is resolved and there seem to be no side effects related to this change, it will now be closed.

          Show
          mason Mason Meyer (Inactive) added a comment - It seems that this issue is functioning as expected. The "ColorBy" feature is applying Heatmap Settings and remembering them in a way that is intuitive to the user. After testing, I can confirm that this story is resolved because the two issues mentioned in this story have been addressed and are working as expected: 1. If I color tracks "A" and "B" with "Heatmap Setting 1", and then color tracks "B" and "C" with "Heatmap Setting 2", track "A" also gets "Heatmap Setting 2" applied when it should remain at "Heatmap Setting 1" 2. If I apply a heatmap setting to a track, and then select that track and another track and apply a different heatmap setting, the heatmap setting that gets remembered is not necessarily the one that is currently applied. Instead, it seems that the track's "individual heatmap setting" is always remembered, not the "group's heatmap setting" Since this issue is resolved and there seem to be no side effects related to this change, it will now be closed.
          Hide
          ieclabau Ivory Blakley (Inactive) added a comment -

          I have rebased this and created a pull request, and reassigned to Dr. Loraine.

          Show
          ieclabau Ivory Blakley (Inactive) added a comment - I have rebased this and created a pull request, and reassigned to Dr. Loraine.
          Hide
          deva Devdatta Kulkarni (Inactive) added a comment -

          I am sorry I have deleted clone from laptop, @Ivory can you do this?
          If you need me, I can do this after going home, I am currently in office.

          Show
          deva Devdatta Kulkarni (Inactive) added a comment - I am sorry I have deleted clone from laptop, @Ivory can you do this? If you need me, I can do this after going home, I am currently in office.
          Hide
          ann.loraine Ann Loraine added a comment -

          Can you (Ivory or Devdatta?) re-base the pull request onto the latest master branch?

          Show
          ann.loraine Ann Loraine added a comment - Can you (Ivory or Devdatta?) re-base the pull request onto the latest master branch?
          Hide
          ieclabau Ivory Blakley (Inactive) added a comment -

          Thanks for the clarification!

          I think this one commit just got lost in our process and we're trying to get it back into place now.
          It's been reviewed by two of us, so I think we can move to the pull request. I'll see if Ann wants us to let Mason take at pass it just to be safe.

          Show
          ieclabau Ivory Blakley (Inactive) added a comment - Thanks for the clarification! I think this one commit just got lost in our process and we're trying to get it back into place now. It's been reviewed by two of us, so I think we can move to the pull request. I'll see if Ann wants us to let Mason take at pass it just to be safe.

            People

            • Assignee:
              mason Mason Meyer (Inactive)
              Reporter:
              deva Devdatta Kulkarni (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: