FLTK logo

STR #3523

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 Home  |  Articles & FAQs  |  Bugs & Features  |  Documentation  |  Download  |  Screenshots  ]
 

Return to Bugs & Features | SVN ⇄ GIT ]

STR #3523

Application:FLTK Library
Status:1 - Closed w/Resolution
Priority:4 - High, e.g. key functionality not working
Scope:3 - Applies to all machines and operating systems
Subsystem:Core Library
Summary:Fl_Menu_Buttons cross over text
Version:1.4.0
Created By:ggarra13
Assigned To:AlbrechtS
Fix Version:1.4.0
Fix Commit:a7c9fd88b3ddc9275be6fa5611ddf60da86b14c3
Update Notification:

Receive EMails Don't Receive EMails

Trouble Report Files:


Name/Time/Date Filename/Size  
 
#1 ggarra13
07:51 May 14, 2019
menus.cpp
1k
 
 
#2 AlbrechtS
06:38 May 15, 2019
patch_v1_debug.diff
5k
 
     

Trouble Report Comments:


Name/Time/Date Text  
 
#1 ggarra13
07:51 May 14, 2019
When a popup Fl_Menu_Button is popped, changes to other Fl_Menu_Buttons effect the original popup menu button and replace its text.  So for example, a popup with File/Open will get replaced with Red Channel from another button.  
 
#2 AlbrechtS
06:38 May 15, 2019
@Gonzalo: Please test 'patch_v1_debug.diff' to see if this fixes the issue for you (it does for me with my test files). Apply with `patch -p0'.

This patch adds one new method to the API, but the test program works w/o modifications since the new method Fl_Menu_::menu_end() is called internally in Fl_Menu_Button::popup(). See also added documentation (work in progress).

Note that I only fixed Fl_Menu_Button::popup() to call menu_end() internally. If this fix is good we can also try to fix Fl_Menu_Bar and Fl_Choice which may experience the same bug (not tested though).

Note also that Fl_Menu_add.cxx contains test/debug statements that can be enabled (see DEBUG_MENU_ADD).

@devs: review and comments appreciated!
 
 
#3 AlbrechtS
07:10 Jun 27, 2019
Fixed in Git repository.

I committed the new method menu_end() and updated the docs accordingly.

I think we can close this STR. Please confirm.
 
 
#4 AlbrechtS
08:45 Jul 03, 2019
Commit 42b8cb7bb821 had a subtle bug in Fl_Menu_::menu_end() which is now hopefully fixed in commit a7c9fd88b3dd.

Thanks to Gonzalo (the OP) for the new test program posted in fltk.general, see:

https://groups.google.com/d/msg/fltkgeneral/7-dmArkVJYI/4YeBTKHpAgAJ

The error I could see was:

  corrupted double-linked list
  Aborted (core dumped)
 
 
#5 AlbrechtS
07:40 Aug 12, 2019
Closed.  
     

Return to Bugs & Features ]

 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.