----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123288/#review78653 -----------------------------------------------------------
Please make sure to use the bug field in reviewboard and CCBUG when committing changes related to bugs. It makes things much simpler to track afterwards. - Aleix Pol Gonzalez On April 7, 2015, 8:50 p.m., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123288/ > ----------------------------------------------------------- > > (Updated April 7, 2015, 8:50 p.m.) > > > Review request for Plasma, David Edmundson, Eike Hein, and Marco Martin. > > > Repository: plasma-desktop > > > Description > ------- > > We only need to sync the changes to kde4 kdeglobals once users changes the > defaults, hence only users KF5 kdeglobals need to be evaluated. > This workarounds (though the change is IMO correct anyway) the kconfig bug, > as we can otherwise pass both local and systemwide kdeglobals into copyTo > method. > > > Diffs > ----- > > kcms/componentchooser/componentchooserbrowser.cpp f2b81ef > kcms/componentchooser/componentchooserterminal.cpp be52334 > kcms/input/mouse.cpp 69759bd > > Diff: https://git.reviewboard.kde.org/r/123288/diff/ > > > Testing > ------- > > W/o the patch i had both > BrowserApplication[$e]=chromium-browser.desktop (local adjustement) > BrowserApplication=firefox.desktop (distro branding) > > in ~/.kde4/share/config/kdeglobals > now i have only locally adjusted value > > > Thanks, > > Hrvoje Senjan > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
