Skip to content

UX: Add navigation menu #50

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
keradus merged 4 commits into from
Feb 6, 2026
Merged

UX: Add navigation menu #50

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
203db2c
Add navigation menu
Kocal Feb 3, 2026
11fba7a
Handle review comments
Kocal Feb 5, 2026
17a2750
Apply suggestion from @keradus
keradus Feb 6, 2026
aa0b6b8
Apply suggestion from @keradus
keradus Feb 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions theme/layout.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,17 @@ <h1 class="title_header">
</header>
{% endblock %}

{%- block navigation %}
Copy link
Copy Markdown
Member

@keradus keradus Feb 5, 2026

Choose a reason for hiding this comment

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

maybe sth like this ?

Image

Copy link
Copy Markdown
Contributor Author

@Kocal Kocal Feb 5, 2026

Choose a reason for hiding this comment

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

WDYT?
Capture d’écran 2026-02-05 à 19 25 59

<nav class="nav">
<ul class="menu">
<li class="{% if pagename == 'doc/usage' or pagename.startswith('doc/usage') %}active{% endif %}"><a href="{{ pathto('doc/usage') }}">Usage</a></li>
Comment thread
Kocal marked this conversation as resolved.
Outdated
<li class="{% if pagename == 'doc/rules/index' or pagename.startswith('doc/rules') %}active{% endif %}"><a href="{{ pathto('doc/rules/index') }}">Rules</a></li>
<li class="{% if pagename == 'doc/ruleSets/index' or pagename.startswith('doc/ruleSets') %}active{% endif %}"><a href="{{ pathto('doc/ruleSets/index') }}">Rule sets</a></li>
<li class="{% if pagename == 'doc/config' or pagename.startswith('doc/config') %}active{% endif %}"><a href="{{ pathto('doc/config') }}">Configuration</a></li>
Copy link
Copy Markdown
Member

@keradus keradus Feb 5, 2026

Choose a reason for hiding this comment

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

overall, love the idea!

can we have those, following user-flow?

  • installation
  • usage
  • config
  • rules
  • sets

Copy link
Copy Markdown
Contributor Author

@Kocal Kocal Feb 5, 2026

Choose a reason for hiding this comment

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

Sure!

</ul>
</nav>
{% endblock %}

{%- block content %}
<main class="bd">
<div class="content">
Expand Down
24 changes: 10 additions & 14 deletions theme/static/css/base.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ a:visited
right: 0;
}

.hd, .bd, .ft
.hd, .nav, .bd, .ft
Comment thread
keradus marked this conversation as resolved.
{
max-width: 1200px;
min-width: 996px;
Expand Down Expand Up @@ -154,36 +154,32 @@ a:visited
border-bottom: 2px dotted #fff;
}

.menu
.nav .menu
{
padding-top: 95px;
width: 940px;
font-family: Arial, sans-serif;
display: flex;
gap: 30px;
padding: 0 15px 40px;
}

.menu li
.nav .menu li
{
color: #ffffff;
font-size: 1.125em;
display: inline;
padding-right: 30px;
}

.menu a
.nav .menu a
{
color: #ffffff;
text-decoration: none;
}

.menu a:hover
.nav .menu a:hover
{
color: #ffffff;
text-decoration: underline;
}

.menu a.active
.nav .menu li.active a
{
color: #444;
font-weight: bold;
Comment thread
keradus marked this conversation as resolved.
Outdated
}

.bd .content
Expand Down