Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 498426 - Breakpoints view: Allow to sort by breakpoints creation time
Summary: Breakpoints view: Allow to sort by breakpoints creation time
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 4.7   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.7 M4   Edit
Assignee: Sarika Sinha CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 498469 509520
  Show dependency tree
 
Reported: 2016-07-25 07:43 EDT by Dani Megert CLA
Modified: 2016-12-20 06:44 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2016-07-25 07:43:41 EDT
We should allow to group breakpoints by age in order to make it easier to disable or delete outdated ones.

This can be done by adding a new group to the Breakpoints view.
Comment 1 Markus Keller CLA 2016-07-26 10:51:22 EDT
> This can be done by adding a new group to the Breakpoints view.

Grouping only allow somewhat arbitrary buckets, e.g.: last 2h, today, yesterday, last 7 days, last month, last year, older

When the "Age" grouping is active, we should also change the sorting of items in the same group, so that most recently touched breakpoints are on top.

"Age" could be defined as "created", "last modified" or "last hit". I think "last hit" would make the view quite noisy, since elements would have to be reordered all the time. "last modified" would also make breakpoints jump around on small modifications like toggling enablement or resetting the hit count.

The creation time is probably the best age indicator.
Comment 2 Sarika Sinha CLA 2016-07-26 20:29:41 EDT
This change request emerged/was updated based on a scientific study on how developers debug, in discussion with Daniel Megert, Markus Keller, Sarika Sinha, and Moritz Beller.
Link to the study  -
http://repository.tudelft.nl/islandora/object/uuid%3Abf3325ce-f246-4977-91bc-785f877347b8?collection=education
Comment 3 Sarika Sinha CLA 2016-11-25 01:35:03 EST
Working on it to provide user an option to select between sorting order ( By Name, By creation time)

Currently Sorting is always By name.
Comment 4 Sarika Sinha CLA 2016-11-29 01:31:36 EST
(In reply to Sarika Sinha from comment #3)
> Working on it to provide user an option to select between sorting order ( By
> Name, By creation time)
> 
> Currently Sorting is always By name.

New Sorting orders -> Name and Age (by Ascending Order)
Comment 5 Eclipse Genie CLA 2016-11-29 04:26:55 EST
New Gerrit change created: https://git.eclipse.org/r/85918
Comment 6 Sarika Sinha CLA 2016-11-30 04:41:16 EST
Confusion on the Sorting Order Conditions - 
1. "Creation Time"  or "Age"
  I am fine with either of them, only point to note that most relevant will on the top. So Age sounds more appropriate but "Creation Time" is easily understandable for the user.
2.  "Default" or "Name" - The current sorting order is not a normal "Name" sorting order as it also depends on the other marker attributes
Comment 7 Dani Megert CLA 2016-11-30 12:49:10 EST
(In reply to Sarika Sinha from comment #6)
> Confusion on the Sorting Order Conditions - 
> 1. "Creation Time"  or "Age"
>   I am fine with either of them, only point to note that most relevant will
> on the top. So Age sounds more appropriate but "Creation Time" is easily
> understandable for the user.

Both are fine, but I would go with 'Creation Time'


> 2.  "Default" or "Name" - The current sorting order is not a normal "Name"
> sorting order as it also depends on the other marker attributes

I'd go for 'Name' as a start.


NOTE: Both entries are missing the mnemonic.
Comment 8 Sarika Sinha CLA 2016-11-30 23:42:26 EST
(In reply to Dani Megert from comment #7)

> 
> NOTE: Both entries are missing the mnemonic.

Problem is how the user will know about Mnemonics in View Menu Item, Pressing "Alt" Just takes away the Menu from Focus. What I see is that "Group By" has "Accelerators" , 1 , 2 etc displayed before the Action Label, If that is ok, I can add that.
Comment 10 Sarika Sinha CLA 2016-12-01 06:31:45 EST
Mnemonics have been added.
General note - Ctrl + F10 opens the view menu with Mnemonics Set.
Comment 11 Sarika Sinha CLA 2016-12-05 03:43:05 EST
Verified using
Eclipse SDK

Version: Oxygen (4.7)
Build id: I20161204-2000