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.

Sign up for GitHub

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

Jump to bottom

Bug: calcite-input-time-picker - when to show ':00' for seconds #2329

Closed
2 tasks
AdelheidF opened this issue Jun 17, 2021 · 2 comments
Closed
2 tasks

Bug: calcite-input-time-picker - when to show ':00' for seconds #2329

AdelheidF opened this issue Jun 17, 2021 · 2 comments
Labels
0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. help wanted Issues that the core team needs help with in a sprint.
Milestone
Sprint 7/19 – 7/30

Comments

@AdelheidF
Copy link

Actual Behavior

It’s not consistent about when it displays the seconds or not. If I just clicked the seconds arrows and set it to 00 it stays on. If I click the arrows for minutes or hours the seconds display goes away. I don’t really care one way or another if seconds is set to 00, but it should be the same in all cases.

image

Expected Behavior

same behavior in all cases

Reproduction Steps or Sample

Relevant Info

Version: @esri/calcite-components@<version>

  • CDN
  • NPM package
@AdelheidF AdelheidF added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Jun 17, 2021
@jcfranco jcfranco added help wanted Issues that the core team needs help with in a sprint. and removed needs triage Planning workflow - pending design/dev review. labels Jul 2, 2021
@jcfranco jcfranco added this to the Sprint 7/19 – 7/30 milestone Jul 2, 2021
@driskull
Copy link
Member

@AdelheidF I'm not able to reproduce this. Do you have a sample?

@eriklharper
Copy link
Contributor

Closing this in favor of a new calcite-input-time component which will solve this issue by showing the seconds field when the step field is less than 60.

#2331 (comment)

@AdelheidF AdelheidF mentioned this issue Aug 3, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. help wanted Issues that the core team needs help with in a sprint.
Projects
None yet
Milestone
Sprint 7/19 – 7/30
Development

No branches or pull requests
4 participants
@jcfranco @eriklharper @driskull @AdelheidF