Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some minor typo/bug fixes, and fix for issue #33 & #67. #71

Open
wants to merge 63 commits into
base: master
Choose a base branch
from

Conversation

FizzyGalacticus
Copy link

Some of the debug statements in RIEStatefulBase were using string template literal syntax, but missed using the tick instead of quote for it to work. Those should now output the intended debugging messages.

Also change the !== comparator in RIEStatefulBase to != to determine whether or not the value of the element had changed. This prevents an unnecessary update function call when the values are the same, just one a number and the other a string.

chilimatic and others added 30 commits May 4, 2017 18:23
Currently, RIEInput :

* defaults its <input> size to 20, which limits visibility of entered
text that's longer.
* only shows 20 characters when an RIEInput field is clicked on, to
enter edit mode.
fixes readme.md changes package json to separate the original package…
Dustin L. Dodson and others added 30 commits March 30, 2018 13:49
…u00A0 as it was altering the word-wrap behavior in an undesired way.
… makes it easier to initialize and handle, and almost all operations on the value convert it to an array anyways. Now, when a new tag is added, the array is converted to a Set and back to an array, to remove duplicates.
CHANGELOG
- Updates dependencies
- Workflow: Basic Jest based tests & Travis integration
- Add: TypeScript definitions
- Add:: Constants for KeyCodes
- Add: formatValue method that formats the getValue() content (note extracted by martinheidegger when debugging)
- Add: consistent debug statements.
- Code Cleanup.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.