all repos — fluxbox @ 11f8d9c21f6da5abf01d942589914b26e04c20d9

custom fork of the fluxbox windowmanager

fix menu rendering bug with selected items
rathnor rathnor
commit

11f8d9c21f6da5abf01d942589914b26e04c20d9

parent

cbc8e23ed2d8f8c3702e8ff53943b5844d1b9de2

2 files changed, 10 insertions(+), 7 deletions(-)

jump to
M ChangeLogChangeLog

@@ -1,5 +1,8 @@

(Format: Year/Month/Day) Changes for 0.9.10: +*04/08/28: + * Fix menu render bug with previously highlighted items (Simon) + FbTk/Menu.cc *04/08/27: * Protect Fluxbox::shutdown better from being double-called (Simon) fluxbox.cc
M src/FbTk/Menu.ccsrc/FbTk/Menu.cc

@@ -22,7 +22,7 @@ // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING

// FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER // DEALINGS IN THE SOFTWARE. -// $Id: Menu.cc,v 1.72 2004/08/03 21:25:51 fluxgen Exp $ +// $Id: Menu.cc,v 1.73 2004/08/28 14:25:52 rathnor Exp $ //use GNU extensions #ifndef _GNU_SOURCE

@@ -695,17 +695,17 @@ MenuItem *tmp = menuitems[which_sub];

tmp->submenu()->internal_hide(); } - if (shown && shown->menu.window == menu.window) - shown = (Menu *) 0; - - torn = visible = false; - which_sub = which_press = which_sub = -1; - // if we have an active index we need to redraw it // as non active int old = m_active_index; m_active_index = -1; drawItem(old, true); // clear old area from highlight + + if (shown && shown->menu.window == menu.window) + shown = (Menu *) 0; + + torn = visible = false; + which_sub = which_press = which_sub = -1; menu.window.hide(); }