osen / fl_flex Goto Github PK

Stumbled across this when using Fl_Flex and a new project.

FL_Round_Buttons will automatically take care of the "radio button" logic [only one button in a set is 'on'] as long as the buttons are in the same Fl_Group.

As the Fl_Flex Row/Columns are derived from FL_Group, this means a set of Fl_Round_Buttons must all be in the same row/column for this automatic behavior to work. Otherwise, the programmer has to manage the button toggling themselves.

Hi!

I was looking at using FL_Flex, seems like good and simple code and quite useful!

It doesn't seem to have a license applied, would you consider applying a license perhaps applying the same FLTK license would be appropriate?

https://github.com/fltk/fltk/blob/master/COPYING

The need to add FL_Box "spacers" between items (e.g. FL_Flex::ROWs) is kinda awkward. Especially when populating a Fl_Flex::COLUMN. Having an auto-spacer setting (e.g. setSpacer(int val)) might be useful. For a column it would add the requested vertical space between children; for a row it would add the requested horizontal space.

I'm liking this class! I nicely re-created a Tkinter GUI of mine pretty quickly:

Note the nice spacing between the Home/Prev/etc buttons. I achieved that by exposing Fl_Flex::margin and setting a value. Works well for a ROW but not so well for a COLUMN.

Warnings from Visual Studio building for 64-bit target:

FL_Flex.cpp(37,28): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(45,22): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(52,27): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(63,27): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(67,55): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(71,91): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(74,23): warning C4267: '+=': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(81,28): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(86,16): warning C4267: '-=': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(89,22): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(96,27): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(107,27): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(111,55): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(120,51): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
FL_Flex.cpp(123,23): warning C4267: '+=': conversion from 'size_t' to 'int', possible loss of data