Community
Participate
Working Groups
Where it occurs: Preferences -> General -> Editors -> Text Editors -> Annotations The "include in next/previous navigation" checkbox isin't alligned with all the widgets above it. I suggest moving all those widgets to the left (see attached pictures).
Created attachment 278859 [details] Suggested fix
Created attachment 278860 [details] Preview of what the suggested fix would look like
It might make more sense to move all of the widgets (which are currently to the right of the table) underneath the table.
Created attachment 278901 [details] scrolled annotation list My annotations list already has scrollbars and instead would profit if the table grow and shrink with the preference window. Beside that my screen has more pixel in width than in height.
(In reply to Andrew Obuchowicz from comment #3) > It might make more sense to move all of the widgets (which are currently to > the right of the table) underneath the table. -1 for that because the distance from the selected annotation to its settings would be bigger.
*** Bug 548059 has been marked as a duplicate of this bug. ***
(In reply to Paul Pazderski from comment #4) > Created attachment 278901 [details] > scrolled annotation list > > My annotations list already has scrollbars and instead would profit if the > table grow and shrink with the preference window. Beside that my screen has > more pixel in width than in height. Thank you for the input Paul, very insightful in helping me approach this problem
(In reply to Dani Megert from comment #5) > (In reply to Andrew Obuchowicz from comment #3) > > It might make more sense to move all of the widgets (which are currently to > > the right of the table) underneath the table. > -1 for that because the distance from the selected annotation to its > settings would be bigger. Good point. As mentioned by Paul, perhaps it'd be better to try and make it so that the table dynamically resizes with the window (and the widgets try to keep their relative positioning)? This could potentially be done using GridLayout.
Comment on attachment 278860 [details] Preview of what the suggested fix would look like Additionally, is there any issue with alligning the widgets with the "Include in next/previous navigation" check box ? (As depicted in the attachment image)
(In reply to Andrew Obuchowicz from comment #9) > Additionally, is there any issue with alligning the widgets with the > "Include in next/previous navigation" check box ? (As depicted in the > attachment image) That sounds OK to me.
New Gerrit change created: https://git.eclipse.org/r/143986
Gerrit change https://git.eclipse.org/r/143986 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=925e3c83512b8467fdbfc8ebb45819c82c18be01
Thanks Andrew.
(In reply to Dani Megert from comment #13) > Thanks Andrew. Thank you for the code review Dani. Should I open a new bug regarding making the table on the Annotations page grow/shrink with the window? (See Paul's comment) I am currently working on another patch but am willing to try and address Paul's issue once I have a moment.
