Summary: | [CSS] Style initially not applied when toolbar is wrapped before user selects it | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Rolf Theunissen <rolf.theunissen> | ||||
Component: | UI | Assignee: | Platform-UI-Inbox <Platform-UI-Inbox> | ||||
Status: | REOPENED --- | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | aobuchow, Lars.Vogel, maidieter, mmarchand, paul-eclipse, twolf | ||||
Version: | 4.11 | ||||||
Target Milestone: | --- | ||||||
Hardware: | PC | ||||||
OS: | Windows 7 | ||||||
See Also: |
https://bugs.eclipse.org/bugs/show_bug.cgi?id=539661 https://bugs.eclipse.org/bugs/show_bug.cgi?id=565754 https://bugs.eclipse.org/bugs/show_bug.cgi?id=576287 |
||||||
Whiteboard: | |||||||
Bug Depends on: | |||||||
Bug Blocks: | 577364 | ||||||
Attachments: |
|
Description
Rolf Theunissen
2019-05-05 15:02:20 EDT
Search view toolbar does not wrap for me anymore. Does is still wrap for you? Not sure if that is a regression or an intended fix. Rolf, please reopen if you still see this issue. Created attachment 283135 [details]
Screenshot of the problem
I still see the issue in I20200602-1800
I still see the issue in I20200602-1800 Mike, please have a look Rolf, do you see the same issue in Dark theme? In the dark theme the issue shows not when changing the toolbar. However, it does show when resizing the search view when the toolbar is wrapped. It disappears again when the view gains or loses focus. I spent some time investigating this but so far I have come up with nothing. BTW: this is an old problem predating 2019. I first ran into this on OS X in 2016 when I gave the EGit history view a search field in the toolbar. I spent quite some time back then investigating this but had been unable to figure out what caused it. In EGit we have some rather gruesome hacks that appear to work around this most of the time. Basically, whenever the search textbox in the toolbar becomes visible (which may cause the toolbar to be wrapped) it focuses first the view's main control and then its own widget. This appears to fix the wrong background most of the time. *** Bug 569022 has been marked as a duplicate of this bug. *** *** Bug 571296 has been marked as a duplicate of this bug. *** Not sure if that helps but a while ago I ported the layout spy to PDE, which allows to assign random colors to UI elements to better identify them and shows additional infos on hover. To trigger: Ctrl+3 -> Layout spy -> Color controls Seems that this is yet another scenario that is left over from Bug 539661 Comment 36 and further. Given the history of that bug, Bug 565754 could have the same root cause as this one. |