Skip to content

Added small UI tweaks#237

Merged
Sp3EdeR merged 1 commit intomasterfrom
feature/small-ui-tweaks
Mar 11, 2026
Merged

Added small UI tweaks#237
Sp3EdeR merged 1 commit intomasterfrom
feature/small-ui-tweaks

Conversation

@Sp3EdeR
Copy link
Owner

@Sp3EdeR Sp3EdeR commented Mar 4, 2026

  • The menu gradient is more subtle in dark mode
  • When the sidebar is narrow, the title text and route date/stars don't wrap any more
  • GitHub is properly formatted

Notes for reviewer:

@Sp3EdeR Sp3EdeR requested review from Csempi, Copilot and nbali March 4, 2026 16:57
@Sp3EdeR Sp3EdeR enabled auto-merge (rebase) March 4, 2026 16:58
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-11 06:26 UTC

@Sp3EdeR Sp3EdeR disabled auto-merge March 4, 2026 16:58
Copy link

Copilot AI left a comment

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 applies small UI refinements to improve dark-mode visuals and prevent sidebar/menu text from wrapping in narrow layouts.

Changes:

  • Adjusted route info header markup to use a non-wrapping flex layout with truncation for the update date.
  • Tweaked dark-mode gradients by separating #menu styling from the general dark background.
  • Fixed “GitHub” capitalization and made the sidebar title truncate instead of wrap.

Reviewed changes

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

File Description
res/huroutes.js Uses Bootstrap flex + truncation utilities to keep route date/stars on one line in narrow sidebar widths.
res/huroutes.css Refines dark-mode appearance by giving #menu its own, subtler gradient background.
index.html Corrects “GitHub” title capitalization and prevents the sidebar header title from wrapping via truncation.
Comments suppressed due to low confidence (1)

res/huroutes.js:606

  • The variable name starsOutter contains a typo (double “t”). Since this line was touched, consider renaming it (and its subsequent uses in this block) to starsOuter to keep naming clear and consistent.
            let starsOutter = $('<span class="stars-outer flex-shrink-0" title="{0}"/>'.format(
                langDict.rating
            )).initTooltip();

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

You can also share your feedback on Copilot code review. Take the survey.

@Sp3EdeR Sp3EdeR force-pushed the feature/small-ui-tweaks branch from f007df5 to aeda603 Compare March 4, 2026 17:48
@nbali nbali requested a review from Copilot March 9, 2026 02:37
Copy link

Copilot AI left a comment

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 2 comments.


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

Copy link

Copilot AI left a comment

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 3 comments.


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

You can also share your feedback on Copilot code review. Take the survey.

* The menu gradient is more subtle in dark mode
* When the sidebar is narrow, the title text and route date/stars don't wrap any more
* GitHub is properly formatted
@Sp3EdeR Sp3EdeR force-pushed the feature/small-ui-tweaks branch from 63566f2 to 54a4062 Compare March 9, 2026 21:22
@nbali nbali requested a review from Copilot March 11, 2026 00:53
Copy link

Copilot AI left a comment

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 no new comments.


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

@Sp3EdeR Sp3EdeR merged commit 5dbe08d into master Mar 11, 2026
8 checks passed
@Sp3EdeR Sp3EdeR deleted the feature/small-ui-tweaks branch March 11, 2026 06:26
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.

3 participants