Community
Participate
Working Groups
It is desirable to add a detail pane to the breakpoints view, see discussion at Bug 360588 comment 3. A proposal will follow to receive comments related to the contents in this detail pane.
Created attachment 237196 [details] screen shots different types of detail panes per break point type A draft implementation for the break point detail pane is now available at https://git.eclipse.org/r/#/c/17181/ you can also see the relevant screen shots attached to this bug in a "Libreoffice Writter" document. The implementation is based on a Mikhail's prototype and was modified to re-use the "common" property page currently available for the break point property dialogue, therefore this implementation is short but dependent on existing page layouts, with the benefit of re-using the saving and validation mechanisms. I.e. Modifying the page layout will also implicate changes to the layout in the property dialogue. Comments and questions are very welcome and expected !
I am not sure it's good idea to re-use the breakpoint's property page as a content of the detail pane. First of all it doesn't fit well in the view and at least needs scroll bars. Having "Condition" and "Ignore count" in the pane is justifiable but I don't see why we need to duplicate the "Enable/disable" option and add the ability of changing the line number. My original patch was intended to display additional context specific information that was hidden from users, like hit counts, pending status and error messages. I still find it more important. Of course, if we modify the breakpoints view to present the corresponding target breakpoints along with the platform breakpoints the detail pane's content of the latter would be similar to what Alvaro is proposing while the target breakpoint's content would include the target related info.
(In reply to Mikhail Khodjaiants from comment #2) > I am not sure it's good idea to re-use the breakpoint's property page as a > content of the detail pane. First of all it doesn't fit well in the view and > at least needs scroll bars. Having "Condition" and "Ignore count" in the > pane is justifiable but I don't see why we need to duplicate the > "Enable/disable" option and add the ability of changing the line number. > My original patch was intended to display additional context specific > information that was hidden from users, like hit counts, pending status and > error messages. I still find it more important. Of course, if we modify the > breakpoints view to present the corresponding target breakpoints along with > the platform breakpoints the detail pane's content of the latter would be > similar to what Alvaro is proposing while the target breakpoint's content > would include the target related info. Thanks for the feedback Mikhail, I see the point on avoiding redundant information, It would be grate if you could provide a small list of the attributes you think are more important to display, were you considering any values that require writing ?
