Closed alexaverill closed 2 weeks ago
Hi @alexaverill this is a really nice addition. I love the character count. I committed some tweaks to the label and hint text to bring them in line with the style elsewhere.
Two requests:
enableKayboard
vs.behavior > enable_keyboard_input
. Can you remove the former? I think you were right to put them in behavior
and snake case is correct—there are only a few legacy keys that are still in camel case.max_char_count
to max_character_count
and save it as an int instead of a string?By the way, I changed your PR to point to the Dev_5.1 branch rather than main.
@forceflow1049 good call outs, I cleaned those up. The first one was just bad initial placement of the defaults. I'll point future PRs towards the latest dev branches!
Adding in the ability of a keyboard to be used for entering text into the Word Cloud. This includes a toggle-able setting during setup, and defaults to off. If this setting is enabled it will hide the on screen keyboard at the moment.
Additionally I thought there would be value in also being able to control the max character length of an entered word if desired. I added in that setting as well, with its default state to not do anything. It also will display a character count to the user if there is one enabled. (Color can be controlled through settings)
This addresses issue #6
Samples: