Opened 15 years ago
Closed 6 years ago
#5293 closed bug (fixed)
Some settings in Appearance preflet are ignored
Reported by: | damoklas | Owned by: | Janus |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Preferences/Appearance | Version: | R1/alpha1 |
Keywords: | Cc: | looncraz | |
Blocked By: | Blocking: | #4868 | |
Platform: | All |
Description
Apperance prefs not work correctly. Etc. menu fonts color – it work for Bezzila, but not work for Haiku apps. And control text color not correct and other bugs.
Attachments (11)
Change History (37)
by , 15 years ago
Attachment: | menutxtaisarisat.png added |
---|
follow-ups: 2 3 comment:1 by , 15 years ago
Which version of Transmission is shown in the screenshot? Is it the Qt based one or the pre 1.0 versions for BeOS/Zeta?
comment:2 by , 15 years ago
comment:3 by , 15 years ago
Replying to mmadia:
Which version of Transmission is shown in the screenshot? Is it the Qt based one or the pre 1.0 versions for BeOS/Zeta?
Qt based: http://qt-haiku.ru/index.php?option=com_rokdownloads&view=file&Itemid=61&id=43:transmission-1
comment:4 by , 15 years ago
Milestone: | R1 → R1/alpha2 |
---|---|
Version: | R1/Development → R1/alpha1 |
Updated info from "haiku-r1a2-rc-rev36542-x86gcc2hybrid".
Settings works in color section:
- Panel background – OK
- Panel text – not OK (see picture1)
- Document background – not OK (see picture2)
- Document text – not OK (see picture3)
- Control background – not OK (see picture4)
- Control text – not OK (see picture5)
- Control border – not OK (see picture6)
- Control highlight – not OK(?) (see picture7)
- Navigation base – OK
- Navigation pulse – not work
- Shine – not OK (?) (see picture8)
- Shadow – not work
- Menu background – OK
- Selected menu item background – not work
- Menu item text – not work (only in BeZillaBrowser works that)
- Selected menu item text – not work
- Selected menu item border – not work
- Tooltip background – OK
- Tooltip text - not work
- Succsess – ?
- Failure – ?
- Window tab – OK
- Window tab text – OK
- Inactive window tab – OK
- Inactive window tab – OK
comment:6 by , 15 years ago
comment:7 by , 14 years ago
Milestone: | R1/alpha3 → R1 |
---|---|
Summary: | Apperance work not correct → Some settings in Appearance preflet are ignored |
This doesn't belong in alpha 3.
by , 13 years ago
Attachment: | Menu Item Background and Text.png added |
---|
Screenshot showing alternative selected menu background color and selected menu text color, I am working on a patch
comment:9 by , 13 years ago
Blocking: | 4868 added |
---|
comment:10 by , 12 years ago
Hey John,
I'd be interested even in a work-in-progress patch which produces the alternative selected menu colors shown in your screenshot.
comment:11 by , 12 years ago
Owner: | changed from | to
---|---|
Status: | new → in-progress |
As an FYI I figured out the code to fix the menu colors and am now fleshing out the various ways colors are hard-coded or otherwise unchangeable. I'm also seeing how dark themes look. Might as well take ownership.
comment:12 by , 12 years ago
Should I commit what I've done pertaining to appearance related fixes or would that get in the way of your in-progress changes for this ticket?
comment:13 by , 12 years ago
They might get in the way. I suspect we fixed a lot of the same things though. But making these changes was useful for me to see just how things work. Could you make a patch and attach it here just so I can see if I missed anything, and just to compare our implementations?
by , 12 years ago
Attachment: | MenuItemSelectedBackgroundColor_1.diff added |
---|
Diff of just the selected menu item background color fix
comment:14 by , 12 years ago
patch: | 0 → 1 |
---|
comment:15 by , 12 years ago
That is my patch for setting the Selected MenuItem Background Color only. I calculated either black or white text based on the background color instead of using the B_MENU_ITEM_TEXT_COLOR constant. I think in general this is a better approach although I can see how using the constant instead could be preferred.
comment:16 by , 12 years ago
OK I think mine is definitely more extensive. For example I'm making use of the setting for the menu text color (normal and selected.) While it would then be possible for a user to configure something unusable (dark on dark or whatever), that is their prerogative.
I also changed things so that the check marks, shortcuts and other menu elements were drawn in that color. I also had to fix BMenuFields to draw correctly with these settings turned on, since it hard codes the control color.
My Haiku time has been limited lately but I'll try to finish this up soon, along with my changes to allow for subpixel text rendering when the properly compiled Freetype is installed.
follow-up: 18 comment:17 by , 12 years ago
Did you get to allowing the ability to set the colors in ListViews (like the one in the Appearance Preferences itself) according to the selected and unselected menu color and text preferences?
comment:18 by , 12 years ago
Replying to jscipione:
Did you get to allowing the ability to set the colors in ListViews (like the one in the Appearance Preferences itself) according to the selected and unselected menu color and text preferences?
As you now probably know, I've committed my menu colors patch in hrev44426, but I did not yet work on making list views follow those color choices. I think it would make some sense though, at least for the selected item.
Right now StringItem (and even the custom list items in Appearances, as you noted in a TODO) hard-code black text. I think what makes the most sense for lists is making use of the document background and document text colors for most of the list, then menu selected colors for the selected items.
comment:19 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | in-progress → assigned |
comment:20 by , 10 years ago
This is the current status of the colors in HAIKU
Color | Status |
---|---|
Panel background | OK |
Panel text | OK |
Document background | OK |
Document text | OK |
Control background | UNUSED |
Control text | MISUSED |
Control border | USED ONLY IN NotificationView notification |
Control highlight | USED IN locale, virtualmemory, diskprobe, notification |
Control mark | OK |
Navigation base | USED AS B_KEYBOARD_NAVIGATION_COLOR and in WorkspaceView |
Navigation pulse | UNUSED |
Shine | USED ONLY in diskprobe (ProbeView.cpp) |
Shadow | USED ONLY in diskprobe (ProbeView.cpp) |
Menu background | OK |
Selected menu item background | OK |
Menu item text | OK |
Selected menu item text | OK |
Selected menu item border | UNUSED |
List background | OK |
Selected list item background | OK |
List item text | OK |
Selected list item text | OK |
Scroll bar thumb | OK |
Tooltip background | OK |
Tooltip text | OK |
Success | USED IN WebPositive (DonwloadProgressView.cpp) |
/haiku/headers/private/app/ServerReadOnlyMemory.h | |
/haiku/src/tests/servers/app/transformation/main.cpp | |
Failure | SAME AS Success + /haiku/src/preferences/time/NetworkTimeView.cpp |
/haiku/src/servers/notification/NotificationView.cpp | |
Window tab | OK |
Window tab text | OK |
Inactive window tab | OK |
Inactive window tab text | OK |
Window border | OK |
Inactive window border | OK |
comment:21 by , 10 years ago
What is the purpose of Control background, Control text, Control border? Now Control text is used in a arbitrary way. Navigation Pulse, Shine, Shadow, Selected menu item border?
comment:22 by , 10 years ago
"Control" colors are supposed to be the default BControl ones. The idea is that instead of changing the color for each individual widget (button, checkbox, etc) we could set a single "control color" setting. BControlLook should then use this as the default color set for all widgets.
The pulse/shine/shadow and selected menu item borders are not used by the current look. An example mockup using more of the colors is here: http://design1st.org/tim/be/images/maui.gif
As you can see, there is a border around the menu selection ("selected menu item border"), all controls share common colors ("control background": light grey, "control text": black, "control border": also black). The navigation pulse/shine/shadow could be used to highlight the keyboard-focused button, for example.
I think the current look does not use these colors directly, and instead computes some shades itself using tint_color (for example to highlight the button under the mouse).
comment:23 by , 10 years ago
Thanks, for the answer...
It would be handy to know which color to use for customer widgets. I fixed some of this glitches in #10840. This problem is noticeable using ThemeManager and Appearance.
comment:25 by , 8 years ago
patch: | 1 → 0 |
---|
comment:26 by , 6 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
I think this is fixed, we can open a specific ticket if we will find some errors in the use of the system colors.
menu txt color