Revert "Improve partial maximization strings"
ClosedPublic

Authored by zzag on Thu, May 14, 5:39 AM.

Details

Summary

This reverts commit 45bf53adab9d998126037134b10492437d42e53e.

We cannot change config values for existing users since it will break
their installations.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Thu, May 14, 5:39 AM
Restricted Application added a project: KWin. · View Herald TranscriptThu, May 14, 5:39 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Thu, May 14, 5:39 AM
zzag added a comment.Thu, May 14, 5:40 AM

Note that the message freeze is today.

broulik added inline comments.
kcmkwin/kwinoptions/kwinoptions_settings.kcfg
80–81

Does it not use the name as config key? i.e. should be fine to change the UI file but not the kcfg?

zzag added inline comments.Thu, May 14, 5:55 AM
kcmkwin/kwinoptions/kwinoptions_settings.kcfg
80–81

Unfortunately, no. The value is written to the config while the name is used for generating C++ enums.

bshah added a subscriber: bshah.Thu, May 14, 6:18 AM

Hm maybe writing kconf_update script could be nicer?

zzag added a comment.Thu, May 14, 6:22 AM

Hm maybe writing kconf_update script could be nicer?

I think changing display strings in the kcm would be much simpler.

davidedmundson accepted this revision.Thu, May 14, 9:36 AM
davidedmundson added a subscriber: davidedmundson.

Heh, I also assume it used name, but judging from options.cpp that's very very clearly not the case. Sorry for not noticing.

This revision is now accepted and ready to land.Thu, May 14, 9:36 AM

Note that the message freeze is today.

It's been like this for N years, whilst I'm happy with the new UI strings, this isn't something we need to rush. Lets revert first fix properly later.

This revision was automatically updated to reflect the committed changes.