hein added a comment.
I'm super sorry, arc patch changed authorship somehow :( It randomly
sometimes does and doesn't do this and I forgot to check
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D15315
To: broulik, #plasma, davidedmundson, fvogt, hein
Cc:
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:3954e7e07c9c: Set gtk-modules to auto-load
appmenu-gtk-module (authored by hein).
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D15315?vs=41316=41565
hein added a comment.
Since we're in the middle of doing tarballs and I know @broulik was anxious
to get this in I'm going to be proactive and land it.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D15315
To: broulik, #plasma, davidedmundson, fvogt, hein
broulik updated this revision to Diff 41316.
broulik added a comment.
- Write into `~/.gtkrc-2.0`, the `GTK2_RC_FILES` variable isn't present for
apps launcher by Plasma or KRunner, for some reason (probably only stuff
launched through klauncher)
REPOSITORY
R120 Plasma Workspace
CHANGES
broulik added a comment.
For some reason `GTK2_RC_FILES` (set by krdb) isn't set here for krunner or
plasmashell so launching apps from there doesn't pick up the
`~/.config/gtkrc-2.0` but only `~/.gtkrc-2.0`.
Krdb sets the launch environment but it's probably not sourced, or too late,
for
broulik updated this revision to Diff 41115.
broulik edited the summary of this revision.
broulik edited the test plan for this revision.
broulik added a comment.
- Support GTK2 too
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
broulik created this revision.
broulik added reviewers: Plasma, davidedmundson, fvogt.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
Makes it work out of the box at least for GTK 3 applications
TEST PLAN
