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

fix window menu placement with toolbar at top of screen, and disable titlebar when doing it

Browse source
This commit is contained in:
markt 2007-01-05 19:58:44 +00:00
parent 416cca0edd
commit dbe528755e
3 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
ChangeLog
View file

@ -1,6 +1,9 @@
(Format: Year/Month/Day)
Changes for 1.0rc3:
*07/01/05:
* Fix placement of window menu with toolbar at the top of the screen,
and remove titlebar from window menu (Mark)
Window.cc IconButton.cc
* Don't change focus to a window that doesn't accept focus (Mark)
FocusControl.cc
* Only grab keybindings on screens managed by fluxbox (Mark)

1
src/IconButton.cc
View file

@ -57,7 +57,6 @@ public:
explicit ShowMenu(FluxboxWindow &win):m_win(win) { }
void execute() {
m_win.screen().hideMenus();
m_win.menu().enableTitle();
// get last button pos
const XEvent &event = Fluxbox::instance()->lastEvent();
int x = event.xbutton.x_root - (m_win.menu().width() / 2);

6
src/Window.cc
View file

@ -2333,8 +2333,10 @@ void FluxboxWindow::showMenu(int menu_x, int menu_y) {
int head = screen().getHead(menu_x, menu_y);
// but not under screen
if (menu_y + menu().height() >= screen().maxBottom(head))
// but not off the screen
if (menu_y < static_cast<signed>(screen().maxTop(head)))
menu_y = screen().maxTop(head);
else if (menu_y + menu().height() >= screen().maxBottom(head))
menu_y = screen().maxBottom(head) - menu().height() - 1 - menu().fbwindow().borderWidth();
if (menu_x < static_cast<signed>(screen().maxLeft(head)))