Opened 18 years ago
Closed 15 years ago
#1089 closed bug (fixed)
Appearance doesn't do anything
Reported by: | jackburton | Owned by: | darkwyrm |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Preferences/Appearance | Version: | R1/pre-alpha1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
If I change the colors using the appearance preflet, nothing happens. The old colors are still used throghout the system. Moreover, if I close and reopen the app, the colors are resetted to the default ones.
Attachments (1)
Change History (9)
comment:1 by , 17 years ago
comment:2 by , 16 years ago
Appearance is somewhat works, I didn't tested all options, but some of them clearly work, btw shouldn't it be renamed to Colors?
comment:3 by , 16 years ago
The bugs in the Appearance preflet should be fixed in hrev27347.
Applying the changes system wide however does not work correctly. 1) Some changes need a repaint request (like scroll bars) 2) Other require restart of the application
I think in R5 was no protocol defined for notification of applications about ui_color changes. So 2 should be the behavior in Haiku.
comment:4 by , 16 years ago
Quick note: The Appearance dialog still does not change the color for window tab's as of hrev28136
by , 16 years ago
Attachment: | patch_decorator added |
---|
comment:5 by , 16 years ago
Regarding the impossibility to change the color of the tabs ..
There was two errors :
- It didn't read the colors from the desktop settings passed in the allocator.. instead it read it from ..??? an other desktop settings instance?
- The Zoom and Close buttons bitmaps didn't store the color it was used to generate them, so changing the color wouldn't trigger a new redraw of them..
My patch fixes those two issues. After that patch, you can change the color in appearance preflet and NEW windows will show the new tab color. However, already opened ones won't change. We need a feedback mecanism to inform the windows that the color scheme changed. We have a similar mecanism to inform them that fonts changed, we might want to copy/adapt it.
comment:7 by , 16 years ago
I applied my patch in hrev30279. I'm not quite sure this ticket should be closed, although, since it covers a wider area than what was fixed with my patch.
comment:8 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
We still can't change the color of an already existing tab.. but that would imply a mecanism to advertise changes in themes/styles/colors/appearance to the applications.. But I think that's outside the scope of this ticket, IMO. I'm closing it, as fixed. Feel free to reopen otherwise.
AFAIK appearance now works a bit more.