Browse Source

Flattened replies: Avoid subtree/sibling confusion

So far the most confusing aspect of the flattened single replies is when
there are sibling comments on the same level. For example, if a comment
has 2 replies (A and B) and A has one reply (C), the flattening would
put all of A, B, and C on the same indentation level as A C B. This was
confusing and made it hard to recognize where a subtree ended and it
went back to sibling comments.

This change makes it so that the flattening will only happen when there
are no siblings. This will reduce how often the flattening is applicable
somewhat, but also eliminates this confusing case entirely.
merge-requests/68/head
Deimos 6 years ago
parent
commit
e9021bde29
  1. 18
      tildes/scss/modules/_comment.scss

18
tildes/scss/modules/_comment.scss

@ -242,16 +242,21 @@
@mixin flatten-single-replies-above-depth($depth) { @mixin flatten-single-replies-above-depth($depth) {
// Makes it so that comments of the specified depth or greater will not indent // Makes it so that comments of the specified depth or greater will not indent
// their replies unless there is more than one reply. Instead, it adds text at the
// top of the next comment (the single reply) indicating that it's a direct reply to
// the above comment.
// their replies when they only have one reply *and* they're also the only reply to
// their parent (this avoids confusion between subtrees and siblings).
// When indenting is avoided, it adds text at the top of the next comment (the single
// reply) indicating that it's a direct reply to the above comment.
// We actually need the selector to check 1 level up from the specified starting
// We actually need the selector to check 2 levels up from the specified starting
// depth because of how we're utilizing nesting. // depth because of how we're utilizing nesting.
$depth: $depth - 1;
@if ($depth < 2) {
@error "Starting depth must be at least 2";
}
$depth: $depth - 2;
.comment[data-comment-depth="#{$depth}"] { .comment[data-comment-depth="#{$depth}"] {
.comment[data-comment-replies="1"] {
.comment[data-comment-replies="1"] > .comment-tree-replies > .comment-tree-item {
& > .comment[data-comment-replies="1"] {
& > .comment-tree-replies { & > .comment-tree-replies {
margin-left: -1px; // compensate for border margin-left: -1px; // compensate for border
@ -268,6 +273,7 @@
} }
} }
} }
}
} }
@include flatten-single-replies-above-depth(4); @include flatten-single-replies-above-depth(4);
Loading…
Cancel
Save