Code review comment for lp:~cimi/overlay-scrollbar/unity-net-workarea-region

Revision history for this message
Andrea Cimitan (cimi) wrote :

but in sanitize_x/y, if the region is empty, usage of region is skipped... so again, won't change much if the xgetwindowproperty fail.

basically we have two solutions if xgetwindowproperty fails:
1) the scrollbars won't take into account the struts (like now, because the region is cleared everytime).
2) the scrollbars will use the region calculated before (if the region will be cleared only inside the if statement).

« Back to merge proposal