Menu

#187 nsDialogs: Hand cursor for link

Stable
closed-accepted
Plugin (47)
5
2008-08-15
2008-06-27
Vorenus
No

Included are the changed full files (along with patch files from the 2.37 release) that add support for the hand cursor when hovering link controls along with RichText box support.

I've also changed the nsDialogs Readme.html detailing the new RichText functions.

Discussion

  • Vorenus

    Vorenus - 2008-06-27
    • assigned_to: nobody --> kichik
     
  • Amir Szekely

    Amir Szekely - 2008-08-15
    • summary: nsDialogs: Hand cursor for link, plus RichText macros --> nsDialogs: Hand cursor for link
    • status: open --> closed-accepted
     
  • Amir Szekely

    Amir Szekely - 2008-08-15

    Logged In: YES
    user_id=584402
    Originator: NO

    Thanks for the patch. I've applied the LINK part of it.

    As for the RichText, there are a few issues that need to be handled first. The free function seems unneeded as WM_SETTEXT copies the text to the window's buffer, the very high text limit is not really required and I fear some versions of richtext might actually try to allocate that much, and finally I want to make sure this works with RTF text. As far as I know, WM_SETTEXT wasn't meant for that. EM_STREAMIN has a special parameter to handle this. Maybe some newer versions of Windows automatically detect RTF files, but older version might not. Please submit this as a new patch and we'll address these issues there.

     

Log in to post a comment.

Oh no! Some styles failed to load. 😵 Please try reloading this page