[email protected] wrote:
> Author: greg.ercolano
> Date: 2009-03-25 16:09:04 -0700 (Wed, 25 Mar 2009)
> New Revision: 6720
> Log:
> o Major mods to Fl_Browser_.{cxx.H}'s doxygen docs
> o Small mods to Fl_Browser.{cxx,H}'s doxygen docs
> o FL_SORT_ASC/DESC -> FL_SORT_ASCENDING/DESCENDING (includes necessary
> mod to test/browser.cxx demo)
>
> TODO: Fl_Browser needs new '\see Related methods: ' that was initiated
> in Fl_Browser_
Great, Greg, thanks!
But one small issue:
> /**
> + Values for has_scrollbar().
> + */
> + enum { // values for has_scrollbar()
> + HORIZONTAL = 1, ///< Only show horizontal scrollbar
> + VERTICAL = 2, ///< Only show vertical scrollbar
> + BOTH = 3, ///< Show both scrollbars (default)
> + ALWAYS_ON = 4, ///< Both scrollbars always on
This looks wrong....................^^^^^^
I don't think that it is "both" here. Isn't it more "if any ..." ?
> + HORIZONTAL_ALWAYS = 5, ///< horizontal scrollbar always on
> + VERTICAL_ALWAYS = 6, ///< vertical scrollbar always on
> + BOTH_ALWAYS = 7 ///< both scrollbars always on
Because BOTH_ALWAYS is 7.
But I didn't look at the code, so I apologize in advance if I'm wrong.
Albrecht
_______________________________________________
fltk-dev mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-dev
