Re: BUG FIX for the Clipboard copy function

Hi Batavo,

On Tue, 20 Aug 2024 16:01:09 +0200
Batavo Silva <[email protected]> wrote:

> Hi Dillo devs!
> Here is just a small bug fix for the Cipboard copy function. The 3rd
> argument passed for the Fl::copy() function should be 1 (not 0) for
> it to copy the input text to the clipboard buffer.
> 
> This issue is found at line 702 of file dillo/dw/fltkplatform.cc:
> [code]
> void FltkPlatform::copySelection(const char *text)
> {
>    Fl::copy(text, strlen(text), 0);  // wrong
> }
> [/code]
> 
> Change it to:
> [code]
> void FltkPlatform::copySelection(const char *text)
> {
>    Fl::copy(text, strlen(text), 1);
> }
> [/code]
> 
> Ref:
> FLTK
> Documentation: https://www.fltk.org/doc-1.3/group__fl__clipboard.html#ga221bc2fa9bd737a8259a94a06f8005f0 

Unfortunately your above change breaks the selection buffer. 

For example, currently when I select some text with the mouse, then
I can middle-click paste that text into any text input box. That no
longer works with your change. 

Regards,
Alex

_______________________________________________
Dillo-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]