FLTK logo

Re: [RFE] STR #3533: Fl_Counter with fixed arrow widths - preliminary code sample

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.bugs  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: [RFE] STR #3533: Fl_Counter with fixed arrow widths - preliminary code sample Albrecht Schlosser Nov 18, 2020  
 
DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR Active]

Link: https://www.fltk.org/str.php?L3533
Version: 1.3-feature


Hi Nicholas, thanks for your comments. I'm aware that the arrow (triangle)
drawing code, particularly that with fl_draw_symbol() is - well, let's say
- suboptimal. But I didn't want to touch it yet.

I've meanwhile been working on a standardization of the "arrow drawing"
code because it's been so different in different widgets that this needed a
standardization before we can "beautify" ;-) it. These changes are now
complete (i.e. working in tests) but need some cleanup before I can and
want to commit these. As I wrote before the current scheme (i.e.
Fl::scheme()) should determine the arrow (and other) drawing functions, but
for now I'm glad to have all arrow/triangle drawing functions standardized
in one place. I'm going to commit these soon and until that's done my work
on Fl_Counter needs to wait. I'll keep you updated.

FTR: the polygon drawing code used in most places will be the new drawing
code used everywhere (rather than fl_draw_symbol in Fl_Counter). Overriding
the code in a particular widget by subclassing will still be possible.


Link: https://www.fltk.org/str.php?L3533
Version: 1.3-feature


Direct Link to Message ]
 
     
Previous Message ]New Message | Reply ]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'.