Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Commit 75b4eef

Browse files
committed
UX: Improve appearance and handling of accepted answers without excerpts
- Hide blockquote for accepted answers without excerpts to avoid unnecessary visual elements. - Adjust title padding for better alignment when excerpts are absent. - Ensure no content fallback (`""`) when excerpts are not provided. - Add `accepted-answer--has-excerpt` class for proper styling differentiation.
1 parent fe6934e commit 75b4eef

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

assets/javascripts/discourse/components/solved-accepted-answer.gjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export default class SolvedAcceptedAnswer extends Component {
2929

3030
get collapsedContent() {
3131
if (!this.hasExcerpt) {
32-
return null;
32+
return "";
3333
}
3434

3535
return htmlSafe(this.acceptedAnswer.excerpt);
@@ -89,6 +89,7 @@ export default class SolvedAcceptedAnswer extends Component {
8989
<PostQuotedContent
9090
class={{concatClass
9191
"accepted-answer"
92+
(if this.hasExcerpt "accepted-answer--has-excerpt")
9293
(unless this.collapsedContent "title-only")
9394
}}
9495
@collapsedContent={{this.collapsedContent}}

assets/stylesheets/solutions.scss

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,15 @@ aside.quote.accepted-answer {
6464
align-items: flex-start;
6565
}
6666

67+
&:not(.accepted-answer--has-excerpt) {
68+
> .title {
69+
padding: 0.8em;
70+
}
71+
> blockquote {
72+
display: none;
73+
}
74+
}
75+
6776
.accepted-answer--solver-accepter {
6877
display: flex;
6978
flex-wrap: wrap;

0 commit comments

Comments
 (0)