D29270: Fix broken localization of kcmkwinrules

victorr added a comment.


  In D29270#659983 <https://phabricator.kde.org/D29270#659983>, @victorr wrote:
  
  > If you rename it to “kcm_kwinrules” with the attached patch, then the translation works.
  >  Now my packages are going to check.
  >  F8271628: source-fix.patch <https://phabricator.kde.org/F8271628>
  
  
  Yes, renaming to “kcm_kwinrules” corrects the display of the translation.
  
  F8271655: Screenshot_20200429_223110.jpeg <https://phabricator.kde.org/F8271655>
  
  In D29270#659987 <https://phabricator.kde.org/D29270#659987>, @victorr wrote:
  
  > In D29270#659983 <https://phabricator.kde.org/D29270#659983>, @victorr wrote:
  >
  > > If you rename it to “kcm_kwinrules” with the attached patch, then the translation works.
  > >  Now my packages are going to check.
  > >  F8271628: source-fix.patch <https://phabricator.kde.org/F8271628>
  >
  >
  > Yes, renaming to “kcm_kwinrules” corrects the display of the translation.
  >
  > F8271655: Screenshot_20200429_223110.jpeg <https://phabricator.kde.org/F8271655>
  
  
  No, not everywhere translation works.
  It took one more patch to the previous one.
  F8272366: kwin-fix.patch <https://phabricator.kde.org/F8272366>
  Before
  
  F8272368: Before.png <https://phabricator.kde.org/F8272368>
  
  After
  F8272372: After.jpeg <https://phabricator.kde.org/F8272372>

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D29270

To: yurchor, #kwin, broulik
Cc: victorr, broulik, pino, ltoscano, zzag, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart