Unity 8

Code review comment for lp:~lukas-kde/unity8/newWindowDecosAndPanel

  1. newWindowDecosAndPanel
  2. Merge into trunk
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

In DesktopStage.qml:

Your updateForegroundMaximizedApp() makes unity8 traverse ApplicationManager model once again unnecessarily. The delegates are already ordered according to ApplicationManager model, so their indexes match. No need to look up for it.

review: Needs Fixing

« Back to merge proposal