Fix RTL Markdown list rendering and spacing (Issue #259)

[rajat12826]

Closes #259

Description:
This PR fixes two rendering discrepancies between the Markdown Viewer and native GitHub Markdown:

1. RTL List Bullets: Added proper padding handling for lists when rendered in RTL ([dir="rtl"]), ensuring that bullet points align securely within the container instead of rendering out of bounds.
2. List-to-Paragraph Spacing: Removed custom margin overrides on .markdown-body ul and .markdown-body ol inside styles.css. This allows the application to naturally inherit GitHub's native 16px margin styling for standard spacing between lists and subsequent block elements (like paragraphs).

Changes Made:

• Removed margin: 0.4em 0; and nested list margins from ul and ol elements in styles.css.
• Added [dir="rtl"] overrides for lists in styles.css to apply padding-right: 2em; and padding-left: 0;.

Testing Performed:

• Verified that LTR lists maintain standard 16px spacing before paragraphs.
• Verified that RTL mode properly aligns bullet points without clipping on the right side.

[vercel]

@rajat12826 is attempting to deploy a commit to the BS Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
markdown-viwer	Ready	Preview	Sep 26, 2026 5:16pm UTC

[Copilot]

Copilot review overview

🔵 Needs a closer look

RTL behavior must cover the split preview, and regression tests are needed.

Review effort: Lite
Findings: 1 · 1

Open (2)

• Regenerate stale desktop CSS resource
• Add RTL list layout regression test

Resolved since last review (2)

• RTL task lists retain incorrect left padding
• RTL task lists retain incorrect left padding

[rajat12826]

The desktop resource is fully synced — both styles.css and desktop-app/resources/styles.css were updated in the same commit a6058e9. npm run build passes locally with "Static build smoke check passed.

[ThisIs-Developer]

The desktop resource is fully synced — both styles.css and desktop-app/resources/styles.css were updated in the same commit a6058e9. npm run build passes locally with "Static build smoke check passed.

It looks like the latest review may be carrying forward findings from an older issue of the PR, even though those points were already addressed. I also cross-checked the current changes with additional review agents, and they did not find any remaining blocking issue.

Based on the current state of the PR, I’m approving and merging it.

Thank you, @rajat12826 for the contribution and especially for responding so quickly to the review feedback and updating the PR along the way.

Thanks again for helping improve Markdown Viewer!