Skip to content

Fix expand/collapse reply preview not showing in some cases - #31639

Merged
t3chguy merged 4 commits into
developfrom
t3chguy/fix/31602
Jan 6, 2026
Merged

Fix expand/collapse reply preview not showing in some cases#31639
t3chguy merged 4 commits into
developfrom
t3chguy/fix/31602

Conversation

@t3chguy

@t3chguy t3chguy commented Jan 5, 2026

Copy link
Copy Markdown
Member

Fixes #31602

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy

t3chguy commented Jan 5, 2026

Copy link
Copy Markdown
Member Author

Can't really get higher test coverage than this with jest/jsdom and having to synthesise clientHeight & scrollHeight not being applied to specific nodes but globally

@t3chguy t3chguy added the Z-Skip-Coverage Skip SonarQube coverage for this PR label Jan 5, 2026
@t3chguy
t3chguy marked this pull request as ready for review January 5, 2026 13:02
@t3chguy
t3chguy requested a review from a team as a code owner January 5, 2026 13:02
Comment thread src/components/views/elements/ReplyChain.tsx Outdated
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy
t3chguy enabled auto-merge January 6, 2026 10:41
@t3chguy
t3chguy added this pull request to the merge queue Jan 6, 2026
Merged via the queue into develop with commit ddad820 Jan 6, 2026
43 of 45 checks passed
@t3chguy
t3chguy deleted the t3chguy/fix/31602 branch January 6, 2026 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Defect Z-Skip-Coverage Skip SonarQube coverage for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expand quotes button not present for some truncated reply-quotes

2 participants