talk icon indicating copy to clipboard operation
talk copied to clipboard

fix off by one error in MAX_REPLY_INDENT_DEPTH

Open marcushaddon opened this issue 3 years ago • 1 comments
trafficstars

What does this PR do?

Fixes bug in which replies to deeply nested comments are incorrectly indented.

What changes to the GraphQL/Database Schema does this PR introduce?

None.

Does this PR introduce any new environment variables or feature flags?

No.

If any indexes were added, were they added to INDEXES.md?

n/a

How do I test this PR?

  1. Enable Flatten Replies
  2. Create a reply chain > 8 comments long
  3. Reply to a reply past the 8th level (at which point all replies should be indented at the same level) and observe that the comment is in line with the comment preceding it (as opposed to the left of it).

marcushaddon avatar Sep 22 '22 21:09 marcushaddon

Deploy Preview for gallant-galileo-14878c canceled.

Name Link
Latest commit 73b360c9e0555c8d2970428d0e107ba2afcd2b1d
Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/632cd443c6e67f00096625de

netlify[bot] avatar Sep 22 '22 21:09 netlify[bot]