-
Notifications
You must be signed in to change notification settings - Fork 17
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
Suggestion: Have a "Save Changes" button on the "Plugin Updater" page #681
Comments
👍 |
@jaswsinc +1 for this. To add, I would like to mention while this thought is up... I've always needed a save changes button at the top of the plugin settings page. I personally loathe having to scroll all the way down to the end of the plugin's settings page to save a change, especially when that setting change is simple as turning the plugin on or off at the top of the page, then having to scroll all the way down to save that change. Thoughts on that? My thoughts on it:
|
@bridgeport Yes, I totally agree. Thank you for posting this! I've noticed the lack of a Save Changes button the Pro Updater plugin to be quite annoying but I kept forgetting to open a GitHub issue so that we could get that fixed. :-) @NoahjChampion I also totally agree with you. I've experienced the same frustration of having to scroll all the way to the bottom to hit the Save All Changes button whenever I'm making changes in a panel at the top. We've discussed a bit internally about revamping the entire plugin options panel UI a bit to organize things better and make things the Save All Changes button more user-friendly, so it's possible this work could be part of that. If we don't get around to a larger UI revamp soon, adding the Save All Changes button to the Plugin Updater page, and at the top of the main options page, would be pretty simple. |
@raamdev ok. This sounds good. Going into a new revamp, not sure what ideas you have about it, wether off a framework or not, but it would also be a better experience I feel if the Panel Headings "plugin-menu-page-panel-heading" would collapse when another panel heading is opened. The content in each panel heading is so detailed focused, I don't think there is much use to have multiple ones open at once? I know for me, there isn't and usually to not make the page extensively long, I have to collapse every panel heading I open so it doesn't make the scroll page longer. This can add a lot of extra unneeded clicks just to avoid scrolling down a long page, which is also (now) because of the only save button being at the bottom of the page. But this also brings up the idea I had of a "Toggle All" or "Open All" and "Close All" option to control the panel headings at once. Thoughts? |
@NoahjChampion That already exists, but you're not the first one to miss that the plugin already has this... @jaswsinc added that part of the UI a long time ago, and I missed that it was even there for the longest time! Thanks for the other feedback. Referencing #96, which is the issue specifically for improving the overall interface. I'll post some mockups on that GitHub issue when we start working on this, so I suggest subscribing to that issue to stay updated--I'd LOVE feedback on any UI changes we propose. :-) |
@raamdev Wow... I never saw that either :( Thanks for that and I will look out for the UI conversations. I'm actually subscribed to everything of Comet Cache :) |
Next Release Changelog:
|
Comet Cache v160416 has been released and includes changes from this GitHub Issue. See the v160416 announcement for further details. This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#681). |
As far as I can tell, there's no way to change save settings (e.g., username, license key, dashboard notifications, beta testers) without also forcing an update.
For instance, going from ZenCache Pro to Come Cache Pro results in a new license key to be updated. When saving the new license key, the Update button will re-download the same installed version, remove the existing, then reinstall.
A save button would resolve this and changes to any of the other minor settings.
Thanks.
The text was updated successfully, but these errors were encountered: