Skip to content

Fix unstyled tip admonition on the customize layouts page - #6329

Open
pasindubalasooriya wants to merge 1 commit into
wso2:masterfrom
pasindubalasooriya:fix-unstyled-tip-add-custom-layouts
Open

Fix unstyled tip admonition on the customize layouts page#6329
pasindubalasooriya wants to merge 1 commit into
wso2:masterfrom
pasindubalasooriya:fix-unstyled-tip-add-custom-layouts

Conversation

@pasindubalasooriya

Copy link
Copy Markdown
Contributor

Purpose

The tip in step 5 of the Add custom layouts section is not styled like the other tips on the page. It renders as an empty tip box followed by an unstyled paragraph.

The admonition content was written at the same indentation as its !!! tip marker:

   !!! tip
   If the server stops working after the changes are added, restart the server before verifying the changes.

Python-Markdown requires the content to be indented four spaces past the marker, so the admonition was parsed with no content and the sentence became a separate plain paragraph. The marker and its content are now indented to match the pattern used by every other admonition in the file.

Live page: https://is.docs.wso2.com/en/7.1.0/guides/branding/add-custom-layouts/#step-3-add-a-custom-layout-design

File changed:

  • en/includes/guides/branding/add-custom-layouts.md

This file is a shared include, so the fix applies to IS 7.0.0, 7.1.0, 7.2.0, 7.3.0 and next. Each of those versions holds only a one line stub that includes this file, so there is no per version copy to update.

I also checked every other admonition in the file and confirmed the tip in step 5 was the only one affected.

Related PRs

None

Test environment

Built IS 7.1.0 locally with its own mkdocs.yml and pinned requirements.txt on Python 3.12, then served the output and inspected the rendered HTML.

  • Build completed with exit code 0 and no errors
  • Both tips on the page now produce identical markup:
<div class="admonition tip"><p class="admonition-title">Tip</p><p>...</p></div>

Security checks

The tip in step 5 of "Add custom layouts" had its content at the same
indentation as the !!! tip marker. Python-Markdown requires the content
to be indented four spaces past the marker, so the admonition rendered
as an empty tip box and the sentence fell out as a plain paragraph
instead of being styled like the other tips on the page.

Indent the marker and its content to match the pattern used elsewhere
in the file.
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: 6a1b17ac-52d1-46c6-b765-594d7fc19968

📥 Commits

Reviewing files that changed from the base of the PR and between 6f14a3b and f408de6.

📒 Files selected for processing (1)
  • en/includes/guides/branding/add-custom-layouts.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The pull request corrects the indentation of the restart tip in the custom layouts branding guide. The tip text remains unchanged.

Changes

Branding documentation

Layer / File(s) Summary
Correct restart tip nesting
en/includes/guides/branding/add-custom-layouts.md
The restart tip content is now correctly nested within the !!! tip admonition.

Change: Other

Merge Risk: ⚪ Minimal · up to f408d

The restart tip now renders consistently with the other styled tips, without changing its content. No current merge-readiness risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: fixing the unstyled tip admonition on the customize layouts page.
Description check ✅ Passed The description includes the required Purpose, Related PRs, Test environment, and Security checks sections. It explains the issue, identifies the affected shared include, documents validation, and add…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Git: CodeRabbit could not clone the repository, so clone-backed analysis was skipped and this review may be incomplete. Verify repository clone access, such as SSH credentials, before requesting another full review. If clone access is intentionally unavailable, use path_filters to narrow the review scope.


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

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