Bug 553841 - Performance: fix repositories view lightweight decorator setup
Summary: Performance: fix repositories view lightweight decorator setup
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 5.6   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 5.7   Edit
Assignee: Thomas Wolf CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-12-05 15:43 EST by Thomas Wolf CLA
Modified: 2019-12-13 15:03 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wolf CLA 2019-12-05 15:43:49 EST
The setup of the lightweight decorator in the git repositories view (and in RepositoryTreeNodeLabelProvider) is not correct; it doesn't work well with the CNF. We've managed to hide the problems mostly by doing our own caching of text decorations, but we have to compute images including their decorations in the label provider on the UI thread instead of doing image decorations in the background decorator. Also, the current setup is too specific for our own decorator and doesn't account for the gitflow decorator. (The gitflow icon overlay on repositories is thus never shown; if it were enabled, it'd blink on refreshes.)

This needs to be fixed and restructured such that the whole setup is more in line with the usual ways. Image decorations should occur in the background decorator, and there should be no blinking. A special problem will be the directories of repositories and working trees shown *after* the decorator.
Comment 1 Eclipse Genie CLA 2019-12-05 17:05:08 EST
New Gerrit change created: https://git.eclipse.org/r/153953
Comment 2 Eclipse Genie CLA 2019-12-13 13:56:53 EST
Gerrit change https://git.eclipse.org/r/153953 was merged to [master].
Commit: http://git.eclipse.org/c/egit/egit.git/commit/?id=b2e98cd3ae4bc45153e5424c331f3cdaa936f537