lp:~mc-return/compiz/compiz.merge-fix771448-desktop-wall-edge-flipping-broken
- Get this branch:
- bzr branch lp:~mc-return/compiz/compiz.merge-fix771448-desktop-wall-edge-flipping-broken
Branch merges
- PS Jenkins bot (community): Approve (continuous-integration)
- Sam Spilsbury: Approve
-
Diff: 440 lines (+138/-128)5 files modifiedplugins/wall/src/wall.cpp (+76/-121)
plugins/wall/src/wall.h (+2/-5)
plugins/wall/wall.xml.in (+4/-1)
src/screen.cpp (+7/-1)
tests/manual/plugins/wall.txt (+49/-0)
Related bugs
Related blueprints
Recent revisions
- 3680. By MC Return
-
wall: Remove mouse polling edge flip detection code, instead turn edges on
in the following situations:1. Edge flip pointer is on (always on)
2. Edge flip move is on, and the screen is grabbed (someone is moving
a window)
3. Edge flip dnd is on, and a dnd type window is mapped (someone is
doing a dnd)Fix a misconstructed boolean condition in removeAction that could cause
actions to be removed twice and as such cause the edge reference count to
go negative.Note that dnd window detection is a bit imprecise, but I didn't want to resort
to using some exotic workarounds for the dnd edge cases (such as non-EWMH
compliant dnd windows which are still compliant with ICCCM). - 3677. By MC Return
-
Removed void WallScreen:
:updateScreenEd geRegions () completely
This is not needed at all, no other plugin needs to re-define the screen
edge regions by itself - just core should do that
The creen edge region updating also confuses the edge flipping functionality
case ConfigureNotify: just breaks out now
Added missing default: case and break (just a style issue) - 3675. By MC Return
-
Wall:
Fixed broken wall edge flipping
Minor cleanup in bool WallScreen::initiateFlip (...) - 3674. By MC Return
-
Grid, code cleanup:
Merged if statements.
Replaced magic number 65535.0f with const float MaxUShortFloat =
std::numeric_limits <unsigned short>::max ();.
Use prefix- instead of postfix-increment.
Removed redundant (float) casts.
Removed redundant brackets.
Added and removed newlines to improve readability.
Minor indentation fixes.Approved by PS Jenkins bot, Sam Spilsbury.
- 3673. By MC Return
-
Animation code cleanup:
Use prefix- instead of postfix- in- and decrements.
Removed redundant brackets.
Added and removed newlines where appropriate.
Fixed indentation.Approved by PS Jenkins bot, Sam Spilsbury.
- 3672. By MC Return
-
Cube Addon .xml cleanup:
Better plugin description.
Uppercase titles.
Improved tooltips and names.Approved by PS Jenkins bot, Sam Spilsbury.
- 3671. By Sam Spilsbury
-
Explicitly check if the smart pointers are null rather than converting
implicitly to bool as boost 1.53 now marks the bool operator as explicit(LP: #1172601). Fixes: https:/
/bugs.launchpad .net/bugs/ 1172601. Approved by PS Jenkins bot, MC Return.
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:compiz/0.9.10