FLTK logo

Re: [MOD] STR #3534: Printing a window may be incorrect with the plastic scheme

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: [MOD] STR #3534: Printing a window may be incorrect with the plastic scheme Albrecht Schlosser Mar 03, 2020  
 
DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR New]

Link: https://www.fltk.org/str.php?L3534
Version: 1.4-current


Regarding my comment in src/Fl_Tiled_Image.cxx: no, the issue is not fixed
by your patch. The comment says "The latter can not be checked here..."
which refers to "... or widget", i.e. we don't have the widget size in this
context.

The documentation above states: "Tiled images can be used as background
images for widgets and windows ..." and "If both \p W and \p H are 0 the
image is repeated as often as necessary to fill the entire window" (which
should, BTW, read "the entire window or widget, respectively).

So, the "trick" is, if both W and H are zero we always fill the entire
*window* and rely on clipping set correctly if only a particular *widget*
should be filled.

The problem arises only if a program calls Fl_Tiled_Image::draw() to fill a
widget and sets W and H to 0 (zero). Well, this is likely a pathological
case. Maybe we should disallow this, but as the documentation says, it's
legal right now.

This is just for clarification. If this is not an issue for you to be
discussed further, feel free to apply your patch and close this STR. TIA.


Link: https://www.fltk.org/str.php?L3534
Version: 1.4-current


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