Issue 418: Improve peak removal UI

issues
Status:closed
Assigned To:Guest
Type:Todo
Area:Skyline
Priority:3
Milestone:3.2
Opened:2015-05-20 13:18 by Brendan MacLean
Changed:2016-07-07 12:08 by Brendan MacLean
Resolved:2016-07-07 12:00 by Kaipo Tamura
Resolution:Fixed
Closed:2016-07-07 12:08 by Brendan MacLean
2015-05-20 13:18 Brendan MacLean
Title»Improve peak removal UI
Assigned To»Brendan MacLean
Type»Todo
Area»Skyline
Priority»3
Milestone»3.2
It was just requested that we add a keyboard shortcut for right-click > Remove Peak > All.

https://skyline.gs.washington.edu/labkey/announcements/home/support/thread.view?entityId=b4567ea5-e143-1032-a4bb-da2025825b36&_anchor=21650#row:21650

Not quite sure how we would advertise this shortcut, as currently all of our shortcuts show up on the main menu.

We should also adjust the UI to more closely fit the way Skyline now works. At the precursor level we should only offer:

right-click > Remove Peak (which does what right-click > Remove Peak > All does today)

Only when Skyline is in single transition viewing mode, and integration changes would change just the single transition, then we should show:

right-click > Remove Peak > <transition-text>
                            All

This will reduce the complexity of performing the most common peak removal operation, which may make the keyboard command less important, but we could certainly do both.

2015-06-22 09:40 Brendan MacLean
Assigned ToBrendan MacLean»kaipot@u.washington.edu
Since you are adding the Apply to All menu item, you can also make this change.

2016-07-07 12:00 Kaipo Tamura
resolve as Fixed
Statusopen»resolved
Assigned Tokaipot@u.washington.edu»Brendan MacLean

2016-07-07 12:08 Brendan MacLean
close
Statusresolved»closed
Assigned ToBrendan MacLean»Guest