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

feat: better sidebar transition #556

Closed
wants to merge 4 commits into from
Closed

Conversation

ulivz
Copy link
Member

@ulivz ulivz commented Jun 7, 2018

Summary

This PR leverages the transition of vue.js official website and also enhances the mobile's style when sidebar slide in from the left side. It hides header, which allows users to focus more on sidebar.

Before:

option-0

Option 1

option-1

Hmmm... the box-shadow looks not very good in this gif, this box-shadow is also migrated from official website. so add a extra static snapshot here:

image

Option 2

option-2

@yyx990803 which one do you prefer?

@ulivz ulivz requested a review from yyx990803 June 7, 2018 18:26
@ulivz ulivz force-pushed the master branch 3 times, most recently from 14d9013 to c7c0cd9 Compare June 8, 2018 19:11
@vuejs vuejs deleted a comment from awulkan Jun 10, 2018
@vuejs vuejs deleted a comment from awulkan Jun 10, 2018
@vuejs vuejs locked and limited conversation to collaborators Jun 10, 2018
@ulivz ulivz force-pushed the master branch 5 times, most recently from c992e38 to c2eaff3 Compare July 12, 2018 18:01
@ulivz ulivz force-pushed the master branch 4 times, most recently from 0c3bc3a to cf1e6fc Compare July 28, 2018 08:12
@ulivz ulivz force-pushed the master branch 2 times, most recently from bd69d41 to 903138e Compare October 3, 2018 22:21
@ulivz ulivz force-pushed the master branch 2 times, most recently from 3f79224 to fb05066 Compare October 23, 2018 15:09
@ulivz ulivz force-pushed the master branch 2 times, most recently from 04f50a4 to edcf194 Compare December 13, 2018 18:21
@ulivz ulivz force-pushed the master branch 5 times, most recently from 316e022 to 1284944 Compare January 29, 2019 11:47
@ulivz ulivz closed this Mar 13, 2019
@ulivz ulivz deleted the better-sidebar-transition branch June 4, 2019 08:21
@ulivz ulivz restored the better-sidebar-transition branch June 4, 2019 08:21
@sarahdayan sarahdayan deleted the better-sidebar-transition branch August 7, 2019 23:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant