After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 773221 - Fixes for win32 theme
Fixes for win32 theme
Status: RESOLVED OBSOLETE
Product: gtk+
Classification: Platform
Component: Themes
3.22.x
Other Windows
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2016-10-19 13:05 UTC by Jaime Velasco
Modified: 2018-05-02 17:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-Fix-tiny-fonts-in-win32-theme (858 bytes, patch)
2016-10-19 13:05 UTC, Jaime Velasco
committed Details | Review
0002-Fix-CSS-warnings-in-win32-theme (876 bytes, patch)
2016-10-19 13:06 UTC, Jaime Velasco
committed Details | Review
0003-win32-theme-Fix-scrollbars (7.50 KB, patch)
2016-10-19 13:07 UTC, Jaime Velasco
committed Details | Review
0004-win32-theme-use-TS_TRUE-in-GetThemePartSize (1.12 KB, patch)
2016-10-19 13:07 UTC, Jaime Velasco
none Details | Review
0005-win32-theme-No-need-to-create-a-DC-for-GetThemePartS (1015 bytes, patch)
2016-10-19 13:08 UTC, Jaime Velasco
none Details | Review
0006-win32-theme-Fix-scale-widget (8.42 KB, patch)
2016-10-19 13:08 UTC, Jaime Velasco
committed Details | Review
0007-gtkwin32theme-delete-if-0-code (933 bytes, patch)
2016-10-19 13:09 UTC, Jaime Velasco
none Details | Review
0008-win32-theme-Fix-popup-window-decorations (970 bytes, patch)
2016-10-19 13:09 UTC, Jaime Velasco
committed Details | Review
0009-win32-theme-Fix-column-headers (1.30 KB, patch)
2016-10-19 13:10 UTC, Jaime Velasco
committed Details | Review
0010-win32-theme-Fix-combobox (4.35 KB, patch)
2016-10-19 13:10 UTC, Jaime Velasco
committed Details | Review
0011-win32-theme-Fix-tooltip-borders (757 bytes, patch)
2016-10-19 13:11 UTC, Jaime Velasco
committed Details | Review
Screenshot of the win32 theme (63.33 KB, image/png)
2016-10-19 13:12 UTC, Jaime Velasco
  Details
win32 theme with these patches (70.40 KB, image/png)
2016-10-19 13:14 UTC, Jaime Velasco
  Details
0001-win32-theme-Fix-levelbar-and-tweak-progressbar (2.63 KB, patch)
2016-11-03 14:25 UTC, Jaime Velasco
none Details | Review
0002-win32-theme-fix-disabled-GtkEntry-style (932 bytes, patch)
2016-11-03 14:26 UTC, Jaime Velasco
none Details | Review
0003-win32-theme-Fix-notebook-tabs (11.35 KB, patch)
2016-11-03 14:26 UTC, Jaime Velasco
none Details | Review
0004-win32-theme-Fix-unfocused-window-buttons (1.82 KB, patch)
2016-11-03 14:26 UTC, Jaime Velasco
none Details | Review
0005-win32-theme-make-separator-visible-again (1.25 KB, patch)
2016-11-03 14:27 UTC, Jaime Velasco
none Details | Review
0006-win32-theme-delete-stray-paren (879 bytes, patch)
2016-11-03 14:29 UTC, Jaime Velasco
none Details | Review
0007-win32-theme-Draw-expander-arrow (1.04 KB, patch)
2016-11-03 14:29 UTC, Jaime Velasco
none Details | Review

Description Jaime Velasco 2016-10-19 13:05:59 UTC
Created attachment 338017 [details] [review]
0001-Fix-tiny-fonts-in-win32-theme

There are lots of brokenness in the win32 theme, These patches try to fix the worst offenders.

They are tested on windows 7, with the default style.
Comment 1 Jaime Velasco 2016-10-19 13:06:39 UTC
Created attachment 338018 [details] [review]
0002-Fix-CSS-warnings-in-win32-theme
Comment 2 Jaime Velasco 2016-10-19 13:07:30 UTC
Created attachment 338019 [details] [review]
0003-win32-theme-Fix-scrollbars
Comment 3 Jaime Velasco 2016-10-19 13:07:52 UTC
Created attachment 338020 [details] [review]
0004-win32-theme-use-TS_TRUE-in-GetThemePartSize
Comment 4 Jaime Velasco 2016-10-19 13:08:20 UTC
Created attachment 338021 [details] [review]
0005-win32-theme-No-need-to-create-a-DC-for-GetThemePartS
Comment 5 Jaime Velasco 2016-10-19 13:08:46 UTC
Created attachment 338022 [details] [review]
0006-win32-theme-Fix-scale-widget
Comment 6 Jaime Velasco 2016-10-19 13:09:10 UTC
Created attachment 338023 [details] [review]
0007-gtkwin32theme-delete-if-0-code
Comment 7 Jaime Velasco 2016-10-19 13:09:34 UTC
Created attachment 338024 [details] [review]
0008-win32-theme-Fix-popup-window-decorations
Comment 8 Jaime Velasco 2016-10-19 13:10:18 UTC
Created attachment 338025 [details] [review]
0009-win32-theme-Fix-column-headers
Comment 9 Jaime Velasco 2016-10-19 13:10:55 UTC
Created attachment 338026 [details] [review]
0010-win32-theme-Fix-combobox
Comment 10 Jaime Velasco 2016-10-19 13:11:22 UTC
Created attachment 338027 [details] [review]
0011-win32-theme-Fix-tooltip-borders
Comment 11 Jaime Velasco 2016-10-19 13:12:18 UTC
Created attachment 338028 [details]
Screenshot of the win32 theme
Comment 12 Jaime Velasco 2016-10-19 13:14:28 UTC
Created attachment 338030 [details]
win32 theme with these patches
Comment 13 Emmanuele Bassi (:ebassi) 2016-10-24 10:18:09 UTC
The patches were all merged into gtk-3-22, but not in master, considering its state of flux.
Comment 14 Jaime Velasco 2016-11-03 14:25:20 UTC
Created attachment 339034 [details] [review]
0001-win32-theme-Fix-levelbar-and-tweak-progressbar

I have a few more patches, as the bug wasn't closed I'll attach them here. If this is not OK for you, tell me and I'll open another bug.

Regards
Comment 15 Jaime Velasco 2016-11-03 14:26:01 UTC
Created attachment 339035 [details] [review]
0002-win32-theme-fix-disabled-GtkEntry-style
Comment 16 Jaime Velasco 2016-11-03 14:26:24 UTC
Created attachment 339036 [details] [review]
0003-win32-theme-Fix-notebook-tabs
Comment 17 Jaime Velasco 2016-11-03 14:26:53 UTC
Created attachment 339037 [details] [review]
0004-win32-theme-Fix-unfocused-window-buttons
Comment 18 Jaime Velasco 2016-11-03 14:27:07 UTC
Created attachment 339038 [details] [review]
0005-win32-theme-make-separator-visible-again
Comment 19 Jaime Velasco 2016-11-03 14:29:12 UTC
Created attachment 339039 [details] [review]
0006-win32-theme-delete-stray-paren

There were a typo in one of the applied patches (0010-win32-theme-Fix-combobox)
Comment 20 Jaime Velasco 2016-11-03 14:29:54 UTC
Created attachment 339040 [details] [review]
0007-win32-theme-Draw-expander-arrow
Comment 21 Daniel Boles 2017-09-03 22:07:12 UTC
(In reply to Jaime Velasco from comment #14)
> I have a few more patches, as the bug wasn't closed I'll attach them here.

Thanks for working on this.

I use GTK+ on Windows a fair amount so am keen to see the win32 theme become more viable, and will try to apply and review these.

(I am fine with Adwaita as I actually prefer my apps to look the same everywhere, but a native theme is nice to have.)


Also, we should probably try to get these all into master, too. I don't think much has changed, or is planned to, on the CSS side there. Testing will show, anyway.
Comment 22 GNOME Infrastructure Team 2018-05-02 17:39:18 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gtk/issues/687.