Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This attempts to address some less-than-ideal names, before such changes would break too many user's code.
scrollbar
,ScrollBar
). Arguably "scrollbar" might be considered a word today, but consistency is important and "progressbar" is definitely not.CheckBox
toCheckButton
(with label) andCheckBoxBare
toCheckBox
(without label). Do the same forRadioBox*
.RadioBoxGroup
toRadioGroup
SetRectMgr
toConfigMgr
: it's a special type used when configuring things (Widget::configure
,Layout::set_rect
,Layout::spatial_nav
). No obvious perfect choice.There are still some widgets with non-typical names where it was decided to keep these:
EditBox
,EditField
(same without the box): other frameworks might call theseInput
,TextInput
,TextEdit
,EditText
,Entry
(there's actually not much consistency here). Some of these names are only for single/multi-line widgets; our widget supports both modes. We also have an associated trait,EditGuard
, which fits well with the current names.TabStack
is a descriptive name: a stack with tabs (compareTabs
, Qt'sTabWidget
or GTK'sNotebook
).DragHandle
might be better namedGrip
or something... in any case, it's not a widely used widget.Finally:
Widget::make_child_id
toWidgetChildren
, which already houses the "reverse" method,find_child_index
. Current widgets with non-default implementations all implementWidgetChildren
directly already.