[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-07 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=413678

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||5.17.3

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-07 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=413678

Nate Graham  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/kde
   ||-gtk-config/308342b16f0a1af
   ||0437e1a7d7049b59d30a81952
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Nate Graham  ---
Git commit 308342b16f0a1af0437e1a7d7049b59d30a81952 by Nate Graham, on behalf
of Mikhail Zolotukhin.
Committed on 07/11/2019 at 17:55.
Pushed by ngraham into branch 'Plasma/5.17'.

Remove gtkrc-2.0 legacy settings

Summary:
Previously GTK KCM wrote to gtkrc redundant settings lines:
# `include "path to chosen gtk2 theme"`
#
```
style "user-font"
{
font-name="whatever font was chosen"
}
widget_style "*" style "user-font"
```

Now GTK KCM does not write those lines to configuration file,
but ones that was kept from previous versions of KCM
are causing various glitches, when applying gtk2
configuration, for theme and font configs respectively.

This patch fixes these issues removing these lines from gtkrc
by force, after applying any configuration from GTK KCM.
Related: bug 413107

Test Plan:
 # Open ~/gtkrc-2.0 and add those lines (theme and font name are arbitrary):
```
include "/usr/share/themes/Adwaita-dark/gtk-2.0/gtkrc"
style "user-font"
{
font_name="Noto Sans Regular"
}
widget_class "*" style "user-font"
```
# Open GTK KCM and apply any configuration
# Reopen ~/gtkrc-2.0 file and check, that those lines were removed

Reviewers: cblack, #plasma, apol, ngraham

Reviewed By: cblack, ngraham

Subscribers: ervin, ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D25147

M  +22   -0src/appearancegtk2.cpp
M  +1-0src/appearancegtk2.h

https://commits.kde.org/kde-gtk-config/308342b16f0a1af0437e1a7d7049b59d30a81952

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread Mikhail Zolotukhin
https://bugs.kde.org/show_bug.cgi?id=413678

Mikhail Zolotukhin  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|ASSIGNED

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread Mikhail Zolotukhin
https://bugs.kde.org/show_bug.cgi?id=413678

Mikhail Zolotukhin  changed:

   What|Removed |Added

   Assignee|aleix...@gmail.com  |zom...@protonmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=413678

--- Comment #2 from mustafazaid2172...@gmail.com ---
Yes if you delete the file and apply the theme again from the system settings
then it will create the file correctly

Here's a pastebin of my current gtkrc-2.0. The bit that's supposed to change is
on the 3rd line, the include path. https://pastebin.com/RCTaiB1m

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread Mikhail Zolotukhin
https://bugs.kde.org/show_bug.cgi?id=413678

--- Comment #1 from Mikhail Zolotukhin  ---
What is the contents of your gtkrc file?

Also, try deleting the gtkrc and then apply settings from the KCM. Will the
gtk2 theme be correctly applied after that?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=413678

Nate Graham  changed:

   What|Removed |Added

 CC||uhh...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 413678] Setting gtk2 theme through system settings doesnt work if a .gtkrc-2.0 file already exists

2019-11-04 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=413678

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org,
   ||zom...@protonmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.