Skip to content

fix(web): render right-aligned Markdown correctly #135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
steipete merged 9 commits into from
Aug 2, 2026
Merged

fix(web): render right-aligned Markdown correctly #135

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
4d833d9
fix(web): render right-aligned Markdown correctly
jjjhenriksen Jul 28, 2026
a8243f4
docs: add right-aligned Markdown proof
jjjhenriksen Jul 28, 2026
417ffc6
fix(web): preserve right-aligned Markdown list structure
jjjhenriksen Jul 29, 2026
59c197a
chore: keep proof in pull request description
jjjhenriksen Jul 31, 2026
515c733
Merge remote-tracking branch 'origin/main' into review/pr-135
steipete Aug 2, 2026
8f90e48
docs: credit right-aligned Markdown fix
steipete Aug 2, 2026
67be0a4
test: open settings before alignment assertion
steipete Aug 2, 2026
4b0a645
test: isolate right-aligned Markdown proof
steipete Aug 2, 2026
ceaa22d
test: allow fractional viewport rounding
steipete Aug 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
## 0.2.2 - Unreleased

- Fixed bot topic endpoints accepting cross-workspace list and create operations by enforcing the token workspace scope. Thanks @ShiroKSH.
- Fixed right-aligned messages rendering Markdown lists and blockquotes outside the message bubble. Thanks @jjjhenriksen.
- Added optional human-readable channel display titles while preserving slug-based routing and uniqueness.
- Added agent-friendly CLI commands for adding and removing message reactions through the existing public APIs, with exact JSON output and terminal-safe human output. Thanks @PollyBot13.
- Removed the redundant "Open in ClickClack" link from embedded channel headers, leaving external navigation to the host's own control.
Expand Down
8 changes: 4 additions & 4 deletions apps/api/internal/webassets/dist/200.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,15 +103,15 @@
<link href="/_app/immutable/chunks/CySd1wef.js" rel="modulepreload">
<link href="/_app/immutable/chunks/DK3Fl9T5.js" rel="modulepreload">
<link href="/_app/immutable/chunks/BuFlayix.js" rel="modulepreload">
<link href="/_app/immutable/entry/app.rLUFGSPX.js" rel="modulepreload">
<link href="/_app/immutable/entry/app.ddMYdzGz.js" rel="modulepreload">
<link href="/_app/immutable/chunks/HclGiUj8.js" rel="modulepreload">
<link href="/_app/immutable/chunks/xihTtKlq.js" rel="modulepreload">
<link href="/_app/immutable/nodes/0.DNzuMmb4.js" rel="modulepreload">
<link href="/_app/immutable/nodes/0.Bdv0azaR.js" rel="modulepreload">
<link href="/_app/immutable/chunks/M3UdkkP6.js" rel="modulepreload">
<link href="/_app/immutable/chunks/DiELhALK.js" rel="modulepreload">
<link href="/_app/immutable/chunks/BEqsQr3A.js" rel="modulepreload">

<link href="/_app/immutable/assets/0.DMQbOJlk.css" rel="stylesheet">
<link href="/_app/immutable/assets/0._gf-j6gS.css" rel="stylesheet">
</head>
<body>
<div>
Expand All @@ -125,7 +125,7 @@

Promise.all([
import("/_app/immutable/entry/start.BJgyZiDJ.js"),
import("/_app/immutable/entry/app.rLUFGSPX.js")
import("/_app/immutable/entry/app.ddMYdzGz.js")
]).then(([kit, app]) => {
kit.start(app, element);
});
Expand Down
1 change: 0 additions & 1 deletion apps/api/internal/webassets/dist/_app/immutable/assets/0.DMQbOJlk.css
View file
Open in desktop

This file was deleted.

1 change: 1 addition & 0 deletions apps/api/internal/webassets/dist/_app/immutable/assets/0._gf-j6gS.css
View file
Open in desktop

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions ...dist/_app/immutable/entry/app.rLUFGSPX.js → ...dist/_app/immutable/entry/app.ddMYdzGz.js
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 ...s/dist/_app/immutable/nodes/0.DNzuMmb4.js → ...s/dist/_app/immutable/nodes/0.Bdv0azaR.js
View file
Open in desktop
File renamed without changes.
8 changes: 4 additions & 4 deletions apps/api/internal/webassets/dist/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,17 +103,17 @@
<link href="./_app/immutable/chunks/CySd1wef.js" rel="modulepreload">
<link href="./_app/immutable/chunks/DK3Fl9T5.js" rel="modulepreload">
<link href="./_app/immutable/chunks/BuFlayix.js" rel="modulepreload">
<link href="./_app/immutable/entry/app.rLUFGSPX.js" rel="modulepreload">
<link href="./_app/immutable/entry/app.ddMYdzGz.js" rel="modulepreload">
<link href="./_app/immutable/chunks/HclGiUj8.js" rel="modulepreload">
<link href="./_app/immutable/chunks/xihTtKlq.js" rel="modulepreload">
<link href="./_app/immutable/nodes/0.DNzuMmb4.js" rel="modulepreload">
<link href="./_app/immutable/nodes/0.Bdv0azaR.js" rel="modulepreload">
<link href="./_app/immutable/chunks/M3UdkkP6.js" rel="modulepreload">
<link href="./_app/immutable/chunks/DiELhALK.js" rel="modulepreload">
<link href="./_app/immutable/chunks/BEqsQr3A.js" rel="modulepreload">
<link href="./_app/immutable/nodes/3.BlUkYxRF.js" rel="modulepreload">
<link href="./_app/immutable/chunks/amAOfu1G.js" rel="modulepreload">

<link href="./_app/immutable/assets/0.DMQbOJlk.css" rel="stylesheet">
<link href="./_app/immutable/assets/0._gf-j6gS.css" rel="stylesheet">
<link href="./_app/immutable/assets/ProductSite.4q0lYJor.css" rel="stylesheet">
</head>
<body>
Expand All @@ -128,7 +128,7 @@

Promise.all([
import("./_app/immutable/entry/start.BJgyZiDJ.js"),
import("./_app/immutable/entry/app.rLUFGSPX.js")
import("./_app/immutable/entry/app.ddMYdzGz.js")
]).then(([kit, app]) => {
kit.start(app, element);
});
Expand Down
72 changes: 72 additions & 0 deletions apps/web/src/styles/messages.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -876,6 +876,78 @@
text-align: right;
}

:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
ul,
:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
ol,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
ul,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
ol {
padding-left: 0;
padding-right: 0;
list-style-position: inside;
Comment thread
jjjhenriksen marked this conversation as resolved.
Comment thread
jjjhenriksen marked this conversation as resolved.
}

:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
li
> ul,
:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
li
> ol,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
li
> ul,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
li
> ol {
padding-left: 0;
padding-right: 22px;
}

:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
li
> p:first-child,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
li
> p:first-child {
display: inline;
}

:root:not([data-message-layout="outlined"])[data-user-align="right"]
.message-group.is-self
.markdown
blockquote,
:root:not([data-message-layout="outlined"])[data-other-align="right"]
.message-group:not(.is-self)
.markdown
blockquote {
padding-left: 0;
padding-right: 12px;
border-left: 0;
border-right: 3px solid var(--accent);
}

:root[data-user-align="right"] .message-group.is-self .attachment-grid,
:root[data-other-align="right"] .message-group:not(.is-self) .attachment-grid {
margin-left: auto;
Expand Down
Loading