Skip to content

Add icons to Project navbar menu and footer links#1116

Merged
yurishkuro merged 6 commits into
jaegertracing:mainfrom
yurishkuro:navbar-project-menu-icons
Jul 9, 2026
Merged

Add icons to Project navbar menu and footer links#1116
yurishkuro merged 6 commits into
jaegertracing:mainfrom
yurishkuro:navbar-project-menu-icons

Conversation

@yurishkuro

@yurishkuro yurishkuro commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Add monochrome Font Awesome icons to each entry in the "Project" navbar dropdown, using a params.icon menu param (consistent with the icon field already used for footer links) instead of raw markup in pre.
  • Swap the boxed bullet glyph (⏺) on the "For Mentees" / "For Mentors" sub-items, which was rendering oddly on some platforms, for a muted em dash (—).
  • Mute the navbar icons and the mentorship sub-item dash with Bootstrap's opacity-50, matching the light/dark theme toggler's icon style.
  • Fix icon-to-text spacing: some Font Awesome free icons (e.g. fa-handshake, fa-comments) use a wider viewBox than most others and overflowed Font Awesome's own .fa-fw fixed-width box, leaving no visible gap before the label. Added a custom, wider, clipped icon box so alignment is consistent regardless of a given icon's native aspect ratio.
  • Add trailing icons to the footer's "Get in touch" list (Via chat, Project video meetings, Report issues on GitHub, Report a security issue).

Test plan

  • Ran hugo server locally and confirmed each icon renders correctly in the Project dropdown
  • Confirmed the mentorship sub-item dash is muted and no longer shows the boxed glyph
  • Confirmed the footer "Get in touch" links render their trailing icons
image image

Replace the plain text entries in the Project dropdown with Font
Awesome icons (monochrome, matching the existing GitHub/social icons)
and swap the boxed bullet glyph on the mentorship sub-items for a
plain bullet character.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings July 9, 2026 14:12
Render an optional trailing icon after each link in the footer's
"Get in touch" list, matching the icon style used elsewhere in the
navbar and footer.

Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro changed the title Add icons to Project navbar menu Add icons to Project navbar menu and footer links Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Jaeger’s Hugo/Docsy theme overrides and site configuration to add Font Awesome icons to “Project” navigation items and the footer “Get in touch” links, while also normalizing the mentorship submenu bullet rendering across platforms.

Changes:

  • Add trailing Font Awesome icons to footer “Get in touch” links via a new optional icon field in params.links.developer.
  • Add leading Font Awesome icons to entries in the “Project” navbar dropdown using Hugo menu pre HTML.
  • Replace the boxed bullet glyph (⏺) with a plain bullet (•) for the “For Mentees” / “For Mentors” submenu items.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
themes/docsy-overrides/layouts/_partials/footer/list.html Renders optional per-link icons in the footer “Get in touch” list.
hugo.yaml Adds icon metadata for footer links and menu pre HTML for Project navbar entries; adjusts mentorship submenu bullet glyph.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Use params.icon on Project navbar menu entries instead of raw <i>
markup in pre, mirroring the icon field already used for the footer
links. Mute both the navbar icons and the mentorship sub-item dash
with Bootstrap's opacity-50, matching the light/dark theme toggler.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings July 9, 2026 14:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.

Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
Comment thread hugo.yaml Outdated
fa-fw assumes icons render at ~1.25em, but fa-handshake and
fa-comments use a 640x512 viewBox and overflow that box, leaving no
visible gap before the label no matter how much padding is added.
Replace fa-fw with a wider, clipped custom box so every icon aligns
consistently regardless of its native aspect ratio.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings July 9, 2026 15:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread themes/docsy-overrides/layouts/_partials/navbar.html
Comment thread themes/docsy-overrides/layouts/_partials/navbar.html
Comment thread themes/docsy-overrides/layouts/_partials/navbar.html
Comment thread themes/docsy-overrides/layouts/_partials/footer/list.html Outdated
Flow-style mappings (params: {...}) were only used on the new menu
entries added in this branch; every other mapping in hugo.yaml is
block-style. Match the file's existing convention.

Signed-off-by: Yuri Shkuro <github@ysh.us>
The template had both a literal space and a ps-1 padding class before
the icon, stacking two gaps instead of one.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings July 9, 2026 15:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@yurishkuro yurishkuro requested a review from Copilot July 9, 2026 15:41
@yurishkuro yurishkuro merged commit 1d4dd74 into jaegertracing:main Jul 9, 2026
8 checks passed
@yurishkuro yurishkuro deleted the navbar-project-menu-icons branch July 9, 2026 15:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

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.

2 participants