DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR New]

Link: http://www.fltk.org/str.php?L2634
Version: 1.4-feature


Oh, OK, I see the changes are pretty large, so a diff is huge.

Nevertheless, I've compiled a zip file:
markcw-Fl_Help_View-orig-vs-new-as-patches.zip
..which contains three directories with Fl_Help_View* files in each:

    orig -- fltk's original src (based on svn id's from markcw's files)
    new  -- markcw's files
    patch -- patch versions of the changes (diff -Naurw)

I found using xdiff on the 'orig' vs 'new' files lets one see
what's changed. (enable "ignore whitespace" flag, as mark's files
seem to have DOS line endings, messing up a straight diff)

I didn't try to make patch versions of mingodad's files,
as that sounded like changes to support markdown, perhaps something
we should support at some point, but perhaps after we apply fixes
just to the html.

We should also re-investigate mingodad's other patch suggestions
that has code simplification, to see if there can be some common
ground between markcw's code and mingodad's cleanup.

BTW: I didn't write Fl_Help_View, but was kinda inspired to
investigate updating it.. but it looks like the changes are quite
large, almost a rewrite.. hard to justify during maintenance releases.

I agree it would be cool if we could get a dillo rewrite.
But honestly, as limited as Fl_Help_View is, it's still quite
useful and 'light' for simple html viewing.. and now that we have
fl_open_uri(), that lets one more easily open a full browser across
all platforms to view html that includes javascript, animations,
videos, etc.


Link: http://www.fltk.org/str.php?L2634
Version: 1.4-feature
Attachment: 
http://www.fltk.org/strfiles/2634/markcw-Fl_Help_View-orig-vs-new-as-patches.zip

_______________________________________________
fltk-bugs mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-bugs

Reply via email to