Skip to content

Commit 2f832b6

Browse files
authored
Changing Sidebar Style Changes (github#6391)
This PR is updating the sidebar so that the current active page sidebar is viewable via bold and the marker.
1 parent 82706fd commit 2f832b6

File tree

2 files changed

+20
-18
lines changed

2 files changed

+20
-18
lines changed

includes/sidebar-specific-product.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,13 +62,13 @@
6262
</ul>
6363
<!-- some categories have no maptopics, only articles -->
6464
{% else %}
65-
<ul class="sidebar-articles list-style-none">
65+
<ul class="sidebar-articles my-2">
6666
{% for article in category[1].articles %}
6767
{% unless article[1].hidden %}
6868
{% capture fullPathToArticle %}{{article[1].href}}{% endcapture %}
6969
{% if article[1].shortTitle %}{% assign articleTitle = article[1].shortTitle %}{% else %}{% assign articleTitle = article[1].title %}{% endif %}
7070
<li class="sidebar-article {% if breadcrumbs.article.href == article[1].href %}active {% if currentPath == fullPathToArticle %}is-current-page{% endif %}{% endif %}">
71-
<a href="{{fullPathToArticle}}" class="pl-4 pr-5 py-1{% if forloop.last %} pb-2{% endif %} color-text-primary">{{ articleTitle }}</a>
71+
<a href="{{fullPathToArticle}}" class="pl-6 pl-4 pr-5 py-1{% if forloop.last %} pb-2{% endif %} color-text-primary">{{ articleTitle }}</a>
7272
</li>
7373
{% endunless %}
7474
{% endfor %}

stylesheets/sidebar.scss

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -37,28 +37,30 @@
3737
// as the browsers are still working on supporting that combination
3838
}
3939

40-
.sidebar-maptopic {
41-
.sidebar-article {
42-
position: relative;
40+
.sidebar-article {
41+
position: relative;
4342

44-
&::before {
45-
content: "";
46-
position: absolute;
47-
left: $spacer-4;
48-
height: 100%;
49-
border-left: 1px solid var(--color-text-primary);
50-
width: 1px;
51-
top: 0;
52-
}
43+
&::before {
44+
content: "";
45+
position: absolute;
46+
left: $spacer-4;
47+
height: 100%;
48+
border-left: 1px solid var(--color-text-primary);
49+
width: 1px;
50+
top: 0;
51+
}
5352

54-
&.active {
55-
&::before {
56-
border-left: 3px solid var(--color-text-primary);
57-
}
53+
&.active {
54+
&::before {
55+
border-left: 3px solid var(--color-text-primary);
5856
}
5957
}
6058
}
6159

60+
.is-current-page > a {
61+
font-weight: bolder;
62+
}
63+
6264
// only display child lists of active elements in sidebar
6365
.sidebar-product.active > ul,
6466
.sidebar-category.active > ul,

0 commit comments

Comments
 (0)