Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Revamped Sidebar #385

Open
wants to merge 4 commits into
base: development
Choose a base branch
from
Open

Revamped Sidebar #385

wants to merge 4 commits into from

Conversation

aswinshenoy
Copy link

  • Collapsible Sidebar
  • Responsive, Scrollbar on y-overflow
  • Added Transparent Logo & Resized the Logo

Screenshots :
image

image

@aswinshenoy aswinshenoy changed the base branch from master to development November 4, 2018 14:14
@chirathr
Copy link
Collaborator

chirathr commented Nov 5, 2018

This looks really nice. Few suggestions:

  • The second design looks better.
  • Remove the shadows from the Login button.
  • Move the Login button and Social media icons to the bottom of the page.
  • Make the X button background same as the page background. Maybe even try removing the border and move it to the right. It looks a bit out of place.
  • The Logo looks a little misaligned. Center it or let it take up the width of the sidebar.

@aswinshenoy
Copy link
Author

  • actually both the designs are the same, the first one is the mobile view, and the second is cropped from a desktop view.
  • There is apparently no shadow on the login button
  • I will remove the 'hamburger' button border, and make it more transparent.
  • I will fix the misaligned logo.

@aswinshenoy
Copy link
Author

As suggested by akash etta, I have also center aligned the sidebar items. I will push this along with some other changes. :)

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.

2 participants