feat(input-time-picker): add separate hour, minute, second and meridiem input fields #11712
+1,520
−1,070
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…or meridiem order after adding the proper rendering order logic for meridiem. Also fixing background color css variable assignment to work with both light and dark modes
…inputs because of issue #3883 and adding meridiem ordering logic, although it doesn't currently work with arabic, so we'll fix that in a follow-up
… for RTL meridiems. This allows the Tab key to go in sequence properly. Need to fix this to match in time-picker too.
…ate localizeTimeStringToParts, updated tests. Refactoring to set the localized input value in willUpdate when the value changes. This will ensure that only the last change to value will relocalize the input value because Lit batches updates to properties in willUpdate.
…nd do some more clean-up
…der ring to match input-text styling, removing unused internal css variables that applied to input-text and getting the toggle icon working to open and close the popover
…g a step argument to toISOTimeString which will force the returned time value to conform to the provided step.
…ction that can be used by parseTimeString and toISOTimeString
…e meridiem setting somehow and there's still a lot of cleanup to do, but this is great progress.
…ding a component.requestUpdate to get the component to re-render when time properties change in the controller.
… removing toISOTimeStringFromParts in favor of just toISOTimeString
…ctored toISOTimeString and isValidTime to accept Time objects with expanded and improved unit tests.
…esses to move from 000 to 001
…ll having trouble re-rendering it on some changes to it
…n't include proper logic to validate fractional seconds
…lharper/2709-input-time-picker-masking
github-actions
bot
added
the
enhancement
… a follow-up issue
…ng a getInputValue test helper function to return a concatenated "input" value. Still have some initial rendering issues with hourFormat, need to decide if a setter for the value property is going to work
…troller setting it directly. This allows the setValue method to work better without the hacky attribute access we had to do in the setter. Need to refactor setValuePart to update each time part in a non-blocking way regardless if the full value is a valid time or not.
…ately clearable. The logic is better than it was, but probably need to match native by clearing the value when the meridiem is deleted.
… on meridiem order
…ping the controller's value property the source of truth
… allow time-picker to eventually use them as well to unify the logic between the 2 components. Next up: seconds and meridiem.
… Restoring the meridiem css class so the test can select the meridiem regardless of its position
…ith revised getInputValue function
|
This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions. |
github-actions
bot
added
the
Stale
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Related Issue: #2709
Summary
This PR adds separate focusable and editable input fields for each portion of the time value in
calcite-input-time-picker.