Skip to content

Enhance guide header layout by adjusting title and subtitle presentation#24

Merged
MrHinsh merged 5 commits intomainfrom
fix-for-trasalted-lists
Jan 20, 2026
Merged

Enhance guide header layout by adjusting title and subtitle presentation#24
MrHinsh merged 5 commits intomainfrom
fix-for-trasalted-lists

Conversation

@MrHinsh
Copy link
Member

@MrHinsh MrHinsh commented Jan 20, 2026

Summary by CodeRabbit

  • New Features

    • Added optional subtitle support to guide headers, allowing for additional context below the main title.
    • Restructured guide header layout with improved visual hierarchy and responsive button alignment.
  • Bug Fixes

    • Resolved a formatting issue in guide header rendering.

✏️ Tip: You can customize this high-level summary in your review settings.

Copilot AI review requested due to automatic review settings January 20, 2026 14:06
@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Walkthrough

The guide header layout was restructured from two equal-width columns to an 8/4 grid split, with the title (upgraded from h2 to h1) and new optional subtitle support occupying the wider column, and action buttons repositioned to the narrower column on medium+ screens. The change introduces a minor defect.

Changes

Cohort / File(s) Summary
Guide header layout restructuring
module/layouts/_partials/components/guide/render-guide.html
Grid layout changed from balanced columns to col-12 col-md-8 / col-12 col-md-4 split. Title element promoted from h2 to h1. Optional subtitle paragraph added beneath title when .Params.subtitle exists. Action buttons repositioned with end-alignment on medium+ screens. ⚠️ Contains stray "d" character appended after left column closing tag (apparent bug).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🔲 Grid columns dance, eight and four,
A title crowned (h1 forevermore),
Subtitles whisper below the fold,
But wait—what's that stray "d"? Rather bold.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: adjusting the header layout from two equal columns to an 8/4 grid split, adding subtitle support, and changing the title element from h2 to h1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-for-trasalted-lists

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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 enhances the guide header layout by improving the title presentation and adding optional subtitle support. The changes make the title more prominent and provide better visual hierarchy.

Changes:

  • Upgraded title from h2 to h1 for proper semantic HTML structure
  • Adjusted column proportions from 6/6 to 8/4, giving more space to the title area
  • Added conditional rendering for an optional subtitle parameter with appropriate styling

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

@MrHinsh MrHinsh added this pull request to the merge queue Jan 20, 2026
Merged via the queue into main with commit 82ccad6 Jan 20, 2026
5 of 6 checks passed
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@module/layouts/_partials/components/guide/render-guide.html`:
- Around line 83-90: Remove the stray "d" text node that follows the closing
</div> in the header block of render-guide.html (the block containing the <h1
class="content-title mb-0">{{ .Title }}</h1> and the optional {{ if
.Params.subtitle }} paragraph); delete that literal character so the markup
returns to valid HTML and nothing extra renders in the header.

Comment on lines +83 to +90
<div class="col-12 col-md-8">
<h1 class="content-title mb-0">{{ .Title }}</h1>
{{- if .Params.subtitle }}
<p class="content-subtitle text-muted fs-5 mt-2">{{ .Params.subtitle }}</p>
{{- end }}
</div>d
<div class="col-12 col-md-4">
<div class="d-flex gap-2 justify-content-md-end">
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove the stray “d” text node.

Line 88 has a literal d after the closing </div>, which will render in the header. That’s a visible defect.

🛠️ Proposed fix
-              </div>d
+              </div>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<div class="col-12 col-md-8">
<h1 class="content-title mb-0">{{ .Title }}</h1>
{{- if .Params.subtitle }}
<p class="content-subtitle text-muted fs-5 mt-2">{{ .Params.subtitle }}</p>
{{- end }}
</div>d
<div class="col-12 col-md-4">
<div class="d-flex gap-2 justify-content-md-end">
<div class="col-12 col-md-8">
<h1 class="content-title mb-0">{{ .Title }}</h1>
{{- if .Params.subtitle }}
<p class="content-subtitle text-muted fs-5 mt-2">{{ .Params.subtitle }}</p>
{{- end }}
</div>
<div class="col-12 col-md-4">
<div class="d-flex gap-2 justify-content-md-end">
🤖 Prompt for AI Agents
In `@module/layouts/_partials/components/guide/render-guide.html` around lines 83
- 90, Remove the stray "d" text node that follows the closing </div> in the
header block of render-guide.html (the block containing the <h1
class="content-title mb-0">{{ .Title }}</h1> and the optional {{ if
.Params.subtitle }} paragraph); delete that literal character so the markup
returns to valid HTML and nothing extra renders in the header.

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.

1 participant