Skip to content

Commit 318d33f

Browse files
committed
address PR review comments
1 parent 6c4c928 commit 318d33f

File tree

2 files changed

+11
-15
lines changed

2 files changed

+11
-15
lines changed

_config.yml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ plugins:
4141

4242
# Blog pagination
4343
paginate_path: "/blog/page:num/"
44-
paginate: 12
44+
paginate: 3
4545

4646
# jekyll-feed
4747
# - our previous feed was called atom.xml so we use it instead
@@ -117,8 +117,4 @@ numfocus:
117117
sponsored_project: https://numfocus.org/project/mdanalysis
118118
donate_button: <a href="https://numfocus.org/donate-to-mdanalysis" target="_blank" style="background:#FF9200;padding:10px;margin:10px 0px;text-align:center;text-decoration:none;font-size:12pt;color:#000000;display:inline-block;border-radius:3px">Donate Now</a>
119119

120-
social:
121-
linkedin: "https://www.linkedin.com/company/mdanalysis/"
122-
bsky: "https://bsky.app/profile/mdanalysis.bsky.social"
123-
discord: "https://discord.com/invite/fXTSfDJyxE"
124-
github: "https://github.com/MDAnalysis/mdanalysis"
120+

_includes/sidebar.html

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,16 +11,16 @@ <h1>
1111

1212
<!-- Social Media Icons -->
1313
<div class="sidebar-social-icons">
14-
<a href="{{ site.social.linkedin }}" target="_blank" title="LinkedIn">
14+
<a href="{{ site.linkedin.url }}" target="_blank" title="LinkedIn">
1515
<i class="fab fa-linkedin"></i>
1616
</a>
17-
<a href="{{ site.social.bsky }}" target="_blank" title="Bluesky">
17+
<a href="{{ site.bluesky.url }}" target="_blank" title="Bluesky">
1818
<img src="{{ site.baseurl }}/public/images/bluesky.svg" width="24" height="24" alt="Bluesky">
1919
</a>
20-
<a href="{{ site.social.discord }}" target="_blank" title="Discord">
20+
<a href="{{ site.discord.invite }}" target="_blank" title="Discord">
2121
<i class="fab fa-discord"></i>
2222
</a>
23-
<a href="{{ site.social.github }}" target="_blank" title="GitHub">
23+
<a href="{{ site.github.repo }}" target="_blank" title="GitHub">
2424
<i class="fab fa-github"></i>
2525
</a>
2626
</div>
@@ -40,22 +40,22 @@ <h1>
4040
The code below dynamically generates a sidebar nav of pages with
4141
`layout: page` in the front-matter. See readme for usage.
4242
{% endcomment %}
43-
43+
4444
{% assign blogpagedir = site.paginate_path | split: ":" | first %}
4545
{% assign blogpagedir_len = blogpagedir | size %}
4646
{% assign pages_list = site.pages | sort: "order" %}
4747
{% for node in pages_list %}
48-
{% if node.title != null%}
49-
{% if node.layout == "page"%}
50-
{% assign prefix = node.url | truncate: blogpagedir_len, '' %}
48+
{% if node.title != null %}
49+
{% if node.layout == "page" %}
50+
{% assign prefix = node.url | truncate: blogpagedir_len, '' %}
5151
{% if prefix != blogpagedir %}
5252
<a class="sidebar-nav-item{% if page.url == node.url %} active{% endif %}" href="{{ node.url }}">{{ node.title }}</a>
5353
{% endif %}
5454
{% endif %}
5555
{% endif %}
5656
{% endfor %}
5757
</nav>
58-
58+
5959
<p>&copy; {{ site.time | date: '%Y' }}. All rights reserved.</p>
6060
</div>
6161
</div>

0 commit comments

Comments
 (0)