-
Notifications
You must be signed in to change notification settings - Fork 510
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
Hide smart update icon when disabled #1031
base: main
Are you sure you want to change the base?
Conversation
Only hide when disabled |
What do you mean? Hide if smart update is disabled regardless of if there's a predicted release date? |
Yes. Otherwise people might be confused why it isn't present for a handful of entries. |
Actually I misunderstood the purpose of the PR. The PR is fine as-id but I want some more opinion on this change. |
Okay, so what you want to do is have the hourglass icon disappear only when there’s no available prediction, right? (Assuming that the 4th smart update parameter is not checked) Otherwise, even with the 4th parameter unchecked, the hourglass icon and predicted time window will still appear like normal. Asking ’cause your wording has me a little confused. Edit: Nevermind, I understood it fine, it seems. |
That's currently how the PR is. I was thinking that the prediction isn't even that reliable so maybe hide it altogether if the 4th parameter is unchecked would be a better idea. But that's up to the mihon people to decide |
Yes, I was thinking about this as well. It would also help users with configuring smart update, as the 4th parameter gives close to no feedback as to what it does. “Predict next release time? Don’t I already have a time window prediction even with it off?” |
Any updates on this? |
It is only hidden when smart update is disabled AND there is no predicted release date.
When smart update is disabled that icon doesn't have much purpose other than maybe checking the prediction, but when there is no prediction it should be hidden.