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

Move "Recently Installed Updates" section to the top. #94

Closed
wants to merge 1 commit into from

Conversation

dmitshur
Copy link
Member

@dmitshur dmitshur commented Oct 28, 2018

This change implements a minor presentation redesign, where the "Recently Installed Updates" section is moved at the top, above the "Updates Available" section. Previously it was at the bottom.

The motivation to do this is to improve the user experience when installing updates from top to bottom. This way, the recently updated package moves up to the section above, which is a much smaller jump than moving it all the way to the bottom of the page.

Clean up some of the code in the process.

Regenerate assets.

Fixes #63. /cc @mvdan

This change implements a minor presentation redesign, where the
"Recently Installed Updates" section is moved at the top, above
the "Updates Available" section. Previously it was at the bottom.

The motivation to do this is to improve the user experience when
installing updates from top to bottom. This way, the recently
updated package moves up to the section above, which is a much
smaller jump than moving it all the way to the bottom of the page.

Clean up some of the code in the process.

Regenerate assets.

Fixes #63.
@dmitshur dmitshur force-pushed the recently-updated-on-top branch from 1af3c3b to 5eb0212 Compare October 28, 2018 21:02
@dmitshur
Copy link
Member Author

dmitshur commented Dec 9, 2018

I've evaluated this approach extensively in my local testing, and did not find it to be an overall improvement to the user experience. I didn't get used to it, even after many weeks. It kept feeling awkward and unfriendly to have the "Recently Updated" section above the "Available Updates".

I can extract the useful implementation cleanups from this change, but not the large UI change. Closing.

@dmitshur dmitshur closed this Dec 9, 2018
@dmitshur dmitshur deleted the recently-updated-on-top branch December 9, 2018 23:23
@dmitshur
Copy link
Member Author

I can extract the useful implementation cleanups from this change

Done in 00930cb.

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.

1 participant