Skip to content

PM-5416: Indent markdown challenge spec lists#7223

Merged
jmgasper merged 1 commit into
developfrom
PM-5416
Jun 22, 2026
Merged

PM-5416: Indent markdown challenge spec lists#7223
jmgasper merged 1 commit into
developfrom
PM-5416

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

What was broken
Markdown-rendered bullet and numbered lists in challenge specifications sat too close to the left edge, making list content look cramped.

Root cause (if identifiable)
The markdown specification component only provided custom table styling and did not add list indentation after the app's global/list reset styles.

What was changed
Added scoped left padding for ordered and unordered lists inside the challenge specification markdown renderer.

Any added/updated tests
No automated tests were added because this is a visual CSS-only spacing change and the existing Jest setup does not assert compiled SCSS layout. Validated with npm run lint, npm test, and npm run build.

What was broken
Markdown-rendered bullet and numbered lists in challenge specifications sat too close to the left edge, making list content look cramped.

Root cause (if identifiable)
The markdown specification component only provided custom table styling and did not add list indentation after the app's global/list reset styles.

What was changed
Added scoped left padding for ordered and unordered lists inside the challenge specification markdown renderer.

Any added/updated tests
No automated tests were added because this is a visual CSS-only spacing change and the existing Jest setup does not assert compiled SCSS layout. Validated with npm run lint, npm test, and npm run build.
@jmgasper jmgasper marked this pull request as ready for review June 22, 2026 04:52
@jmgasper jmgasper requested a review from kkartunov as a code owner June 22, 2026 04:52
@jmgasper jmgasper merged commit 5fde6d2 into develop Jun 22, 2026
5 of 6 checks passed
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