nilix/fluxbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixes a little bug with snapping to windows which have a border only.

Browse source 
such windows dont have DECORM_ENABLED and hence no borderwidth was 
added to the calculation. thats fixed now
This commit is contained in:
mathias 2005-04-18 15:27:59 +00:00
parent 6c1288b6f5
commit ebad1ce12f
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/Window.cc
View file

@ -3328,11 +3328,12 @@ void FluxboxWindow::doSnapping(int &orig_left, int &orig_top) {
// we only care about the left/top etc that includes borders // we only care about the left/top etc that includes borders
int borderW = 0; int borderW = 0;
if (decorationMask() & DECORM_ENABLED) if (decorationMask() & (DECORM_ENABLED|DECORM_BORDER|DECORM_HANDLE))
borderW = frame().window().borderWidth(); borderW = frame().window().borderWidth();
int top = orig_top; // orig include the borders int top = orig_top; // orig include the borders
int left = orig_left; int left = orig_left;
int right = orig_left + width() + 2 * borderW; int right = orig_left + width() + 2 * borderW;
int bottom = orig_top + height() + 2 * borderW; int bottom = orig_top + height() + 2 * borderW;
@ -3386,7 +3387,8 @@ void FluxboxWindow::doSnapping(int &orig_left, int &orig_top) {
if ((*it) == this) if ((*it) == this)
continue; // skip myself continue; // skip myself
bw = (*it)->decorationMask() & DECORM_ENABLED ? (*it)->frame().window().borderWidth() : 0; bw = (*it)->decorationMask() & (DECORM_ENABLED|DECORM_BORDER|DECORM_HANDLE) ?
(*it)->frame().window().borderWidth() : 0;
snapToWindow(dx, dy, left, right, top, bottom, snapToWindow(dx, dy, left, right, top, bottom,
(*it)->x(), (*it)->x(),