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

Add autorefresh button to scheduling checks page #3185

Merged
merged 2 commits into from
Dec 24, 2020
Merged

Conversation

willgearty
Copy link
Member

@willgearty willgearty commented Dec 11, 2020

This adds a button to the scheduling checks page that, when clicked, causes all of the checks to refresh at some interval that is specified by the user (default is 30 seconds). When you click the button again, the refreshing stops.

image

image
(My dev server was turned off when I took these screenshots, hence the failed loading)

Fixes #3175.

Copy link
Contributor

@kkbrum kkbrum left a comment

Choose a reason for hiding this comment

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

Looks good, but I'm not sure it's abundantly clear to a first time user what the number to the left of auto refresh is. Could you either add hover text to the input field to say that its the number of seconds between refreshes, or add the word "seconds" to the right of the input field? It's probably easy enough to guess the meaning, but I'd rather people didn't have to guess.

Copy link
Contributor

@kkbrum kkbrum left a comment

Choose a reason for hiding this comment

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

My bad, there is hover text, you just have to hover longer than I expected! Good to go!

@willgearty willgearty merged commit 4b2a464 into main Dec 24, 2020
@willgearty willgearty deleted the sched-auto-refresh branch December 24, 2020 14:21
willgearty added a commit that referenced this pull request Dec 29, 2020
willgearty added a commit that referenced this pull request May 27, 2021
* Initial docs for stable release 13

* Docs for #3116, #3117, and #3118

* Added docs about django upgrade

* Docs for #3128

* Docs for #3129, #3133, #3134, and #3137

* Docs for #3156 and #3153

* Docs for #3174, #3163, and #3184

* Docs for #3139, #3140, and #3141

* Docs for #3143, #3150, #3154, #3160, #3162, and #3168

* Docs for #3171, #3185, #3186, and #3188

* Docs for #3131 and #3189

* Docs for #3149 and #3190

* Docs for #3193, #3194, #3195, #196, and #3197

* Clarification

* Docs for #3192, #3201, #3209, and #2248

* Docs for #3204, #3212, #3214, #3205, 9fd073c, and #3226

* Docs for #3232, de5861c, #3231, #3233, #3234, #3237, #3238, and #3239

* Fix indent

* Docs for #3227 and #3235

* Add missing word

* spelling

* Docs for e57581f, #3255, #3253, #3257, and #3249

* Docs for #3254, #3260, and #3262

* Docs for #3263, #3272, #3240, #3264, #3266, and #3270

* clarifications

* Docs for #3283 and #3252

* Docs for #3288 and misc commits

* Docs for #3292, #3311, #3286, #3289, and #3279

* Docs for a377f0d; move note

* Docs for #3315, #3290, and #3322

* Docs for #3273 and #3317

* Final edits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto-refresh button/form for scheduling checks
2 participants