New component TextFieldEditorWithPopup - #6023
bubblobill wants to merge 5 commits into
Conversation
FlatIconColors. Added missing Icons.WARNING Added Icons.ACTION_EDIT_IN_EDITOR Plus generic directional triangle icons. New class TextFieldEditorWithPopup.java Implemented in TokenPropertiesManagementPanel Usual i18n keys added/changed
kwvanderlinde
left a comment
There was a problem hiding this comment.
I like the new component! I just have a couple requests after doing some tests.
Previously, in Campaign Properties > Token Properties, the cells in the Default columns would automatically switch to a text field on the first click. Now, it either requires a double-click, or to start typing while the cell has focus. Can we restore the single-click behaviour?
I also noticed that after opening and closing the text editor, the popout button gains a yellow background. It also happens if I click and drag the button, and possibly in other circumstances. It doesn't seem to mean anything, though, so would be nice to remove that.

Should be doable. The colour thing is something in my focus handling. Will fix. |
Removed some spurious code.
|
@kwvanderlinde Sorted. Fixed the colour hanging around when not wanted and single click to edit. |
Identify the Bug or Feature request
resolves #6022
Description of the Change
Needed some better icons, so;
and
Usual i18n keys added/changed
Removed TextFieldEditorWithPopupTableCellEditor
Added TextFieldEditorWithPopup
TextFieldEditorWithPopup is a container for:
Label and Toggle button are both optional and not normally shown, e.g.

Enabling them allows use in other situations such as a toolbar;

Has method to get text field with button as a CellEditor.
Has method to open the component in an input dialogue.
Should work as an Accessibile Component
Possible Drawbacks
none
Documentation Notes
n/a
Release Notes
n/a
This change is