FLTK logo

[master] 36af6a5 - Solves Fl_Tree focus box artifacts on linux - fixes issue #299.

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 
 All Forums  |  Back to fltk.commit  ]
 
Previous Message ]Next Message ]

[master] 36af6a5 - Solves Fl_Tree focus box artifacts on linux - fixes issue #299. "Greg Ercolano" Nov 27, 2021  
 
commit 36af6a57295a5f3c674134cab60dc644c96e97fa
Author:     Greg Ercolano <erco@seriss.com>
AuthorDate: Sat Nov 27 10:20:47 2021 -0800
Commit:     Greg Ercolano <erco@seriss.com>
CommitDate: Sat Nov 27 10:20:47 2021 -0800

    Solves Fl_Tree focus box artifacts on linux - fixes issue #299.
    
    Nabbed the current code from Fl_Widget::draw_focus() which solves.

 src/Fl_Tree_Item.cxx | 29 +++++------------------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git src/Fl_Tree_Item.cxx src/Fl_Tree_Item.cxx
index 9861127..da66a21 100644
--- src/Fl_Tree_Item.cxx
+++ src/Fl_Tree_Item.cxx
@@ -796,6 +796,7 @@ Fl_Tree_Item *Fl_Tree_Item::find_clicked(const Fl_Tree_Prefs &prefs, int yonly)
 }
 
 static void draw_item_focus(Fl_Boxtype B, Fl_Color fg, Fl_Color bg, int X, int Y, int W, int H) {
+  // Pasted from Fl_Widget::draw_focus(); we don't have access to this method
   if (!Fl::visible_focus()) return;
   switch (B) {
     case FL_DOWN_BOX:
@@ -807,32 +808,12 @@ static void draw_item_focus(Fl_Boxtype B, Fl_Color fg, Fl_Color bg, int X, int Y
     default:
       break;
   }
-  fl_color(fl_contrast(fg, bg));
-
-//#if defined(USE_X11) || defined(__APPLE_QUARTZ__)
-  fl_line_style(FL_DOT);
-  fl_rect(X + Fl::box_dx(B), Y + Fl::box_dy(B),
-          W - Fl::box_dw(B) - 1, H - Fl::box_dh(B) - 1);
-  fl_line_style(FL_SOLID);
-/*#else
-  // Some platforms don't implement dotted line style, so draw
-  // every other pixel around the focus area...
-  //
-  // Also, QuickDraw (MacOS) does not support line styles specifically,
-  // and the hack we use in fl_line_style() will not draw horizontal lines
-  // on odd-numbered rows...
-  int i, xx, yy;
-
   X += Fl::box_dx(B);
   Y += Fl::box_dy(B);
-  W -= Fl::box_dw(B) + 2;
-  H -= Fl::box_dh(B) + 2;
-
-  for (xx = 0, i = 1; xx < W; xx ++, i ++) if (i & 1) fl_point(X + xx, Y);
-  for (yy = 0; yy < H; yy ++, i ++) if (i & 1) fl_point(X + W, Y + yy);
-  for (xx = W; xx > 0; xx --, i ++) if (i & 1) fl_point(X + xx, Y + H);
-  for (yy = H; yy > 0; yy --, i ++) if (i & 1) fl_point(X, Y + yy);
-#endif*/
+  W -= Fl::box_dw(B)+1;
+  H -= Fl::box_dh(B)+1;
+  fl_color(fl_contrast(fg, bg));
+  fl_focus_rect(X, Y, W, H);
 }
 
 /// Return the item's 'visible' height. Takes into account the item's:
Direct Link to Message ]
 
     
Previous Message ]Next Message ]
 
 

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'.