zzag requested changes to this revision.
zzag added a comment.
This revision now requires changes to proceed.


  In D23415#530822 <https://phabricator.kde.org/D23415#530822>, @GB_2 wrote:
  
  > Is someone from #kwin <https://phabricator.kde.org/tag/kwin/> finally ok 
with this?
  
  
  Nope, nothing has changed since last time.
  
  In D23415#521193 <https://phabricator.kde.org/D23415#521193>, @ngraham wrote:
  
  > Martin F once gave me a very valuable piece of advice: stay within your 
area of expertise.
  
  
  I agree with that. I usually tend to look into code changes rather than UI 
changes. However, KWin developers have total right to veto your [VDG's] patches 
if you change for example the name of an effect, a script, a placement policy, 
etc.
  
  In D23415#521439 <https://phabricator.kde.org/D23415#521439>, @ngraham wrote:
  
  > In fact, the HIG already says that: ...
  
  
  "Smart" is not an arbitrary string, like "left button click," etc.
  
  ---
  
  Requesting changes because this patch still renames smart placement policy. 
The only way to proceed further from this point is to leave the name of smart 
placement policy as is.

REPOSITORY
  R108 KWin

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

To: ngraham, #kwin, #vdg, ndavis, filipf, zzag
Cc: broulik, hein, hpereiradacosta, GB_2, alexde, davidedmundson, romangg, 
cfeck, filipf, zzag, ndavis, kwin, kde-doc-english, LeGast00n, 
The-Feren-OS-Dev, sbergeron, jraleigh, gennad, fbampaloukas, mkulinski, 
ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, 
alexeymin, skadinna, himcesjf, lesliezhai, ali-mohamed, hardening, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to