Skip to content
This repository has been archived by the owner on Mar 8, 2020. It is now read-only.

#2327 - docs navigation changes to improve usability #2717

Merged
merged 3 commits into from
Nov 15, 2017

Conversation

KaiUsher
Copy link
Contributor

Reworked the docs navigation in order to make it much more usable.

Issue #2327

Signed-off-by: K. L. Usher kai.usher@ibm.com

@KaiUsher KaiUsher force-pushed the issue-2327-new branch 2 times, most recently from a25a4d4 to 1cd039f Compare November 15, 2017 11:09
@nklincoln nklincoln added the P1 label Nov 15, 2017
@@ -3,7 +3,7 @@
{% if page.index-order %}
{% capture mods %}{{ page.index-order | modulo:100 }}{% endcapture %}
{% if mods == "0" and page.exception == nil %}
- [{{ page.title }}](..{{page.url}}.html)
- [<b>{{ page.title }}</b><img src="/composer/unstable/assets/img/chevron-left.svg" class="caret">](..{{page.url}}.html)
Copy link
Contributor

Choose a reason for hiding this comment

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

unstable?

Signed-off-by: K. L. Usher <kai.usher@ibm.com>
&:after {
content: " ";
position: absolute;
top: 0;
right: 0px;
bottom: 0;
width: 2px;
// width: 2px;
Copy link
Contributor

Choose a reason for hiding this comment

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

delete?

@@ -167,7 +166,7 @@ $callout: #FFD640;
top: 0;
right: 0px;
bottom: 0;
width: 2px;
// width: 2px;
Copy link
Contributor

Choose a reason for hiding this comment

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

delete?

@@ -124,20 +124,20 @@ $callout: #FFD640;
top: 0;
right: 0px;
bottom: 0;
width: 2px;
// width: 2px;
Copy link
Contributor

Choose a reason for hiding this comment

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

delete?

@winslet
Copy link
Contributor

winslet commented Nov 15, 2017

Needs to be reviewed by design before merging plz!

@nklincoln nklincoln merged commit 7ad3db6 into hyperledger-archives:master Nov 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

4 participants