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

fix relative time picker minus click #4173

Merged
merged 3 commits into from
Nov 30, 2023
Merged

fix relative time picker minus click #4173

merged 3 commits into from
Nov 30, 2023

Conversation

monicawoj
Copy link
Contributor

Clicking minus too many times (when the value was very small) caused an error to occur. This PR resolves this issue by disabling the minus button in the relative time picker when the current value is below the minimum default values (in other words, below 1m).

Copy link

alwaysmeticulous bot commented Nov 29, 2023

🤖 Meticulous spotted visual differences in 8 of 370 screens tested: view and approve differences detected.

Last updated for commit 3911481. This comment will update as new commits are pushed.

Copy link
Contributor

@yomete yomete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@manojVivek manojVivek merged commit 5da9dfb into main Nov 30, 2023
38 checks passed
@manojVivek manojVivek deleted the time-picker-bug branch November 30, 2023 11:01
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.

None yet

3 participants