Re: findbar behaviour

On Wed, Dec 26, 2012 at 11:29:50PM +0000, corvid wrote:
> 
> Here's what resulted, but it seems relatively inelegant:
> [snipped long patch]

  It's  much  simpler. In this case you can let FLTK do the focus
change among widgets:


diff -r 20a26b8bb366 src/findbar.cc
--- a/src/findbar.cc    Wed Dec 26 23:31:48 2012 +0100
+++ b/src/findbar.cc    Thu Dec 27 15:32:02 2012 -0300
@@ -158,7 +158,6 @@ Findbar::Findbar(int width, int height)
     check_btn  =  new Fl_Check_Button(x, border, 2*button_width,
height,
                               "Case-sensitive");
     x += 2 * button_width + gap;
-    check_btn->clear_visible_focus();
    add(check_btn);

 }

  and that's it, forward with Tab, back with Shift+Tab.


  Now, if there's a need for something more special, like letting
Tab alone cycle around, adding this will do:


@@ -179,6 +178,10 @@ int Findbar::handle(int event)
    if (event == FL_KEYBOARD && modifier == 0 && k == FL_Escape) {
       /* let the UI handle it */
       return 0;
+   } else if (event == FL_KEYBOARD && modifier == 0 && k == FL_Tab) {
+      MSG("Findbar::handle FL_KEYBOARD Tab\n");
+      (Fl::focus() == i) ? check_btn->take_focus() : i->take_focus();
+      return 1;
    }

    return Fl_Group::handle(event);


-- 
  Cheers
  Jorge.-