all repos — fluxbox @ a6146af44f3ef1f44894d5545f449f46db8a3c79

custom fork of the fluxbox windowmanager

fix placement issue with xinerama (only hit if screens are vertically aligned)
simonb simonb
commit

a6146af44f3ef1f44894d5545f449f46db8a3c79

parent

f6140840f85e35354c219e3a505c6d1aeb4d3837

2 files changed, 6 insertions(+), 2 deletions(-)

jump to
M ChangeLogChangeLog

@@ -1,6 +1,10 @@

(Format: Year/Month/Day) Changes for 1.0rc2: *06/06/19: + * Fix menu placement issue with (vertical) xinerama (Simon) + - note, doesn't fix "submenus open on wrong screen" issue, that requires + some more serious data movement. + FbTk/Menu.cc * Fix bug #1507671, clock not updating on multiple screens (Simon) - time() and gettimeofday() are inconsistent, only use gettimeofday - clean up Timer stuff a bit, fixed some other bugs
M src/FbTk/Menu.ccsrc/FbTk/Menu.cc

@@ -1075,8 +1075,8 @@ menu.x_shift = 0; //-theme().borderWidth();

m_shifted = true; } - if (y() + height() + 2*theme().borderWidth() > m_screen_height) { - menu.y_shift = m_screen_height - height() - 2*theme().borderWidth(); + if (y() + height() + 2*theme().borderWidth() > m_screen_y + m_screen_height) { + menu.y_shift = m_screen_y + m_screen_height - height() - 2*theme().borderWidth(); m_shifted = true; } else if (y() + (signed) theme().titleHeight() < 0) { menu.y_shift = 0; // -theme().borderWidth();;