-
Notifications
You must be signed in to change notification settings - Fork 80
feat(input-time-picker): add separate hour, minute, second and meridiem input fields #11712
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
Draft
eriklharper
wants to merge
68
commits into
dev
Choose a base branch
from
eriklharper/2709-input-time-picker-masking
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,752
−1,115
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
… 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. |
…tion to updating the TimeController's internal properties and only updating the component's value in the special cases of 1: initial mount and 2: when the time-picker's value changes. Also fixing a test bug with the getInputValue helper function.
…lharper/2709-input-time-picker-masking
…ly when it is actually changing and only update localized properties when the value isn't actually changing (syncs up the localized with the non-localized properties)
…nges since the useT9N controller manages the language. Relocalizes the display on lang change, but need to update the hourFormat when the lang changes too. Going to try moving property change logic to the controller too so that this logic can be shared by both time picking components. Improving the getInputValue test helper
…onent lang changes
…lharper/2709-input-time-picker-masking
…lharper/2709-input-time-picker-masking
…s for this to each locale and skipping broken 12/24 hour format tests to simplify the CI a bit while I work on fixing those tests to type correctly in the new input masks.
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
.