Skip to content

Add collapsible sidebar#258

Merged
dgarcia360 merged 7 commits intoscylladb:masterfrom
dgarcia360:add-collapsible-sidebar
Dec 8, 2021
Merged

Add collapsible sidebar#258
dgarcia360 merged 7 commits intoscylladb:masterfrom
dgarcia360:add-collapsible-sidebar

Conversation

@dgarcia360
Copy link
Copy Markdown
Collaborator

Closes #193

I moved the selector to the bottom and inside the sidebar to not have conflicts with the scrollbar or the multiversion selector.

How to test this PR

  1. Clone this PR. For more information, see Cloning pull requests locally.

  2. Enter the docs folder, and run:

make preview
  1. Open http://127.0.0.1:5500/ with your favorite browser, and enter an internal page.

The sidebar collapsed should look like:

image

The sidebar expanded should look like:

image

@dgarcia360 dgarcia360 requested a review from tzach December 8, 2021 08:35
@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Dec 8, 2021

Two points to consider:

  1. The Previous and Collapse buttons look the same
    image

  2. The Button is not always visible, hidden behind the footer
    image

@dgarcia360
Copy link
Copy Markdown
Collaborator Author

The Previous and Collapse buttons look the same

We can try with another icon, for example:

image

I'll keep you posted once I update the PR.

The Button is not always visible, hidden behind the footer

Good catch! Fixed with the latest commit, please run make clean before running again make preview.

@dgarcia360
Copy link
Copy Markdown
Collaborator Author

dgarcia360 commented Dec 8, 2021

@tzach Collapsible button updated with round borders and new icon.

Preview:

image
image

Copy link
Copy Markdown
Collaborator

@tzach tzach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dgarcia360 dgarcia360 merged commit dd9efa8 into scylladb:master Dec 8, 2021
@dgarcia360 dgarcia360 deleted the add-collapsible-sidebar branch December 8, 2021 15:41
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.

Collapsible navigation

2 participants