> On May 29, 2014, 7:10 a.m., Ben Cooksley wrote: > > Code wise, this change looks fine. In terms of renaming the desktop files - > > i'm fine with changing the filenames, but please don't change the name of > > the application itself. Ideally the KF5 based system settings will still be > > able to set configuration details relevant for KDE 4 applications. > > Hrvoje Senjan wrote: > >Ideally the KF5 based system settings will still be able to set > configuration details relevant for KDE 4 applications. > > that sounds great to me - just that it would need work in every kcm > module. additional caveats: > what if there's only 4.x variant of the kcm? > an option is removed in KF5 variant? > most importantly - how to implement it (different config locations, etc)? > =) > > Ben Cooksley wrote: > Indeed, that could complicate things quite a bit. I'm not sure what we > should do in that case then - but we can't have two "System Settings" > applications installed which do different things on the same system.... > > Martin Gräßlin wrote: > it's kind of the same situation as we had during the KDE3 -> 4 > transition. We had kcontrol to configure KDE3 and systemsettings to configure > KDE4. > > The big problem for the developers is that we don't have any access to > the KDE4 configuration files, thus it's difficult to adjust the > configuration. Now to make it even more complex: should adjusting e.g. the > widget style in a Plasma Next session really change the widget style of a > Plasma 4 session or just in Plasma Next? > > Ben Cooksley wrote: > I'm guessing that KDE4 applications probably won't be able to make full > use of Plasma Next themes, assuming they keep full compatibility (elements > over time may get new names, change format, etc). > > The problem in this case is that the applications have the same name. > Should we come up with a new name for the Plasma Next series, or does a > mechanism exist to only show particular *.desktop files under Plasma Next / > KDE4? > > In any case, the changes to rename the binary and the filenames of the > *.desktop files themselves can go ahead - they're independent of this > discussion. > > Hrvoje Senjan wrote: > >In any case, the changes to rename the binary and the filenames of the > *.desktop files themselves can go ahead - they're independent of this > >discussion. > so it's a ship it, as is now? ;-) > > Ben Cooksley wrote: > Yes, it is. > > Hrvoje Senjan wrote: > are we too late for Plasma/5.0 branch? i guess so...
As this improves co-installability, and System Settings is often referenced for user support i'd appreciate being able to have both KDE 4 and Plasma 5 based System Settings installable if possible. I've no objections to this going into the Plasma/5.0 branch - unless release-team / distributions do... - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118388/#review58690 ----------------------------------------------------------- On May 28, 2014, 7:32 p.m., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118388/ > ----------------------------------------------------------- > > (Updated May 28, 2014, 7:32 p.m.) > > > Review request for Plasma and Ben Cooksley. > > > Repository: systemsettings > > > Description > ------- > > while workspace might not be targeted to co-exist with 4.x variant - > systemsettings should IMHO be able to co-exist. not only workspace components > are adjusting in there, and telling people to do kcmshell$notinstalledvariant > $wantedkcm is very user-unfriendly... > one TODO if this gets a green light, is to rename desktop files, so people > know which variant they are opening. > > > Diffs > ----- > > app/systemsettings.desktop 5f27318 > app/kdesystemsettings.desktop 946d498 > app/CMakeLists.txt c45f7e7 > > Diff: https://git.reviewboard.kde.org/r/118388/diff/ > > > Testing > ------- > > > Thanks, > > Hrvoje Senjan > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
