Community
Participate
Working Groups
the following are some errors found during the UI review that need to be investigated: Properties: - Review Level - Tab "All" - the list of properties is not complete - The path string is not shown, check it for file contexts and files - Formal Review: The class field was not editable Dialogs: - The OK button seems enabled even if there is no valid input Design Rules: - The properties shall match the ones for the group, however one refers to the folder the other one to the file
Adding one more: Navigator: - Editing the description of a comment does not refresh the label in the navigator but after closing and opening the review
Some answers (In reply to comment #0) > the following are some errors found during the UI review that need to be > investigated: > > Properties: > - Review Level - Tab "All" - the list of properties is not complete > - The path string is not shown, check it for file contexts and files > - Formal Review: The class field was not editable These are fixed > > Design Rules: > - The properties shall match the ones for the group, however one refers to > the folder the other one to the file To be consistent, the group should display the group file, not the folder
(In reply to comment #1) > Adding one more: > Navigator: > - Editing the description of a comment does not refresh the label in the > navigator but after closing and opening the review Comments should not be editable once they are created
New additions after discusssing UI review internally among the team. These are high-priority issues to be resolved before next release is out - Remove "Link with Editor" function in Review Navigator. The function should be commented out and refacter as a full sync later on. - Make all properties R4E properties view->All property tab read-only for now - Change R4E Properties view to have only 2 tabs: List (old All) tab and Edit tab - Use expandable composites in Edit tab to reduce footprint in view - Remember folders when triggering ad folder dialogs - New anomaly dialog: Deisng rules should be shown in searchable table like in legacy R4E - Add support to add reply to a Comment - In formal reviews, prevent phase change when email is not sent out - Disable mouse button scroll in all combo boxes - Include icon legen in help - When right-clicking in empty area of the review navigator, show default context-menu (not based on selected tree element) - New review dialog: Do not put any default review type in combo box - All New xxx dialogs: do not enable OK button until minimal information is entered - Always focus on newly created element in Review navigator tree - Always display all options in send notification type dialog - Fix scrollbars in R4E Properties view that are broken when resizing the view - All text boxes should wrap in R4E porperties view - Add command to configure the Groups/Rules sets from the review navigator (show appropriate properties) - Remove Link with porperties command from review navigator (make it always enabled) Lower priority tasks: -Split review navigator view to show elements below the FileContext in separate table pane -Save tree expansion state beofre using refresh command and restore it afterwards
>> >> Design Rules: >> - The properties shall match the ones for the group, however one refers to >> the folder the other one to the file >To be consistent, the group should display the group file, not the folder This is now inmplemented both REview Group and Rule Sets show the file path in their properties
Here's the current status on this item. IMPLEMENTED for July release: - Remove "Link with Editor" function in Review Navigator. The function should be commented out and refacter as a full sync later on. - Make all properties R4E properties view->All property tab read-only for now - New review dialog: Do not put any default review type in combo box - Remove Link with porperties command from review navigator (make it always enabled) - Always focus on newly created element in Review navigator tree - Disable mouse button scroll in all combo boxes - In formal reviews, prevent phase change when email is not sent out - All New xxx dialogs: do not enable OK button until minimal information is entered OUTSTANDING for July release: - Change R4E Properties view to have only 2 tabs: List (old All) tab and Edit tab - Use expandable composites in Edit tab to reduce footprint in view - Include icon legen in help OUTSTANDING as future improvements: - Remember folders when triggering ad folder dialogs - New anomaly dialog: Deisng rules should be shown in searchable table like in legacy R4E - Add support to add reply to a Comment - Always display all options in send notification type dialog - Fix scrollbars in R4E Properties view that are broken when resizing the view - All text boxes should wrap in R4E porperties view - Add command to configure the Groups/Rules sets from the review navigator (show appropriate properties) Potential future improvements: -Split review navigator view to show elements below the FileContext in separate table pane -Save tree expansion state beofre using refresh command and restore it afterwards Will not be implemented - When right-clicking in empty area of the review navigator, show default context-menu (not based on selected tree element). --> This should not be implemented. We now behave like any other navigator and changing the code will break that.
Update: IMPLEMENTED for July release: - Include icon legen in help
All outstanding issues for the July release are now done. What's left: OUTSTANDING as future improvements: - Remember folders when triggering ad folder dialogs - New anomaly dialog: Design rules should be shown in searchable table like in legacy R4E - Add support to add reply to a Comment - Always display all options in send notification type dialog - Fix scrollbars in R4E Properties view that are broken when resizing the view - All text boxes should wrap in R4E porperties view - Add command to configure the Groups/Rules sets from the review navigator (show appropriate properties) Potential future improvements: -Split review navigator view to show elements below the FileContext in separate table pane -Save tree expansion state beofre using refresh command and restore it afterwards
Shall not be in Assigned state if assigned to default r4e-triaged
Mylyn has been restructured, and our issue tracking has moved to GitHub [1]. We are closing ~14K Bugzilla issues to give the new team a fresh start. If you feel that this issue is still relevant, please create a new one on GitHub. [1] https://github.com/orgs/eclipse-mylyn
