Ubuntu
compiz-plugins-main package

Merge lp://qastaging/~elvisd79/ubuntu/oneiric/compiz-plugins-main/fix-for-772177 into lp://qastaging/ubuntu/oneiric/compiz-plugins-main

  1. Oneiric (11.10)
  2. fix-for-772177
  3. Merge into oneiric
Proposed by Elvis Dominguez
Status: Work in progress
Proposed branch: lp://qastaging/~elvisd79/ubuntu/oneiric/compiz-plugins-main/fix-for-772177
Merge into: lp://qastaging/ubuntu/oneiric/compiz-plugins-main
Diff against target: 27 lines (+8/-1)
2 files modified
debian/changelog (+7/-0)
staticswitcher/staticswitcher.xml.in (+1/-1)
To merge this branch: bzr merge lp://qastaging/~elvisd79/ubuntu/oneiric/compiz-plugins-main/fix-for-772177
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Disapprove
Robert Roth (community) Approve
Ubuntu branches Pending
Review via email: mp+65740@code.qastaging.launchpad.net

Description of the change

set "Static Application Switcher/Default Popup Delay" option's default value to 0 instead of 0.2 (LP: #772177)

To post a comment you must log in.
Revision history for this message
Robert Roth (evfool) wrote :

Looks fine.

review: Approve
Reply
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Thanks for your contribution there.

However, as I already explained in another merge request asking for exactly the same change, setting the value to 0 is not the solution. Indeed, if you alt + tab between two applications very quickly, you will then see a flickering dialog appearing. The delay is there to prevent that and it was discussed in natty as part of a design decision.

Alt + Tab is still slow for other reason, not really linked to this 0s.

You should have a look at the other branch linked to the same bug that I rejected for the same reason. You have more explanation there as well :)
Thanks nevertheless, but think about reading all the available information please ;)

review: Disapprove
Reply

Unmerged revisions

14. By Elvis Dominguez

* staticswitcher/statisswitcher.xml.in
  - set "Default Popup Delay" option's default value to 0 instead of 0.2 (LP: #772177)

13. By Elvis Dominguez

Changed Static application Switcher delay to 0
LP:#772177

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches