#3925 closed defect (fixed)
Layout code in Home View does not respect an Alert
| Reported by: | erikos | Owned by: | erikos |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | Sugar | Version: | Git as of bugdate |
| Severity: | Unspecified | Keywords: | r+, regression |
| Cc: | humitos | Distribution/OS: | Fedora |
| Bug Status: | Assigned |
Description
- open an Alert in the Home View (e.g. use the register option in the owner icon Palette)
---> the Owner icon stays at the same position but the ring icons shift down by the height of the alert.
Attachments (2)
Change History (13)
comment:1 Changed 10 years ago by erikos
comment:2 Changed 10 years ago by godiard
- Priority changed from Unspecified by Maintainer to Normal
comment:3 Changed 10 years ago by humitos
- Cc humitos added
- Keywords r? added
I found the problem here:
That line removes the allocation.y value to the y position from where we are going to draw the buddy. So, with or without an Alert message this result will be always the same and the y position of the it will be the same.
We should remove just the height of the main toolbar. A patch with the hardcoded size value is attached to see the difference. It doesn't look good, but it's an approach.
Changed 10 years ago by humitos
An approach to this fix. The value is hardcoded and that doesn't look good :)
comment:4 Changed 10 years ago by erikos
- Keywords r- added; r? removed
So there are two approaches here:
a) when the Alert appears, shift down all the icons in the Home View (owner icon to activity icons)
b) when the Alert appears, leave all icons at the same place
In the old shell we did do (b). So, I think preferred is to keep that behaviour if not too hard to do.
comment:5 Changed 10 years ago by erikos
- Keywords regression added
Changed 10 years ago by humitos
Leave the icons at the same place as erikos said. This patch just works for the RingLayout
comment:6 Changed 10 years ago by godiard
- Keywords r? added; r- removed
comment:7 Changed 10 years ago by manuq
- Keywords r- added; r? removed
I think we need to investigate a more general approach.
comment:8 follow-up: ↓ 9 Changed 10 years ago by godiard
Manuq, Can you explain a little more what you don't like about the solution proposed by humitos?
comment:9 in reply to: ↑ 8 Changed 10 years ago by manuq
- Keywords r+ added; r- removed
Replying to godiard:
Manuq, Can you explain a little more what you don't like about the solution proposed by humitos?
I took another look and I think humitos solution is the right one. In the owner icon allocation we do the same adjustments, and is OK to calculate the spiral using the total screen height (allocation.height + allocation.y).
comment:10 Changed 10 years ago by erikos
- Resolution set to fixed
- Status changed from new to closed
Great, pushed the patch with small adjustments to the description and the in code comment. Pushed to the master and sucrose-0.98 branch.
see #3827 for inspiration.