New Post Page Followups

[micahmo]

Bug Description

This issue is for tracking additional work that needs to be done to the new post/comment page as a followup to #1363 in order to bring it in parity with the old one. The new one is still behind an experimental flag, so these things aren't super urgent, but should be done before releasing it for general availability.

Known Issues (these are things intentionally deferred from the initial implementation)

• Bring back FAB
• Bring back comment navigation
  • Allow comment navigation to navigate between top level comments, and also child comments
  • Ensure that the first navigation goes to the top of the first comment (see Re-introduce comment navigator for new post page #1496 (comment))
• Bring back comment search - with the ability to scroll to specific child comments

Additional Issues (these have been found while daily driving the new page)

• Collapsing comment threads does not show any animation (for the child comments being hidden)

Details

qemu-system-x86_64_G9JKLLDJq0.mp4

• Collapsing a comment thread with only deferred child comments does not hide the placeholder (this is especially confusing because if you try to load the deferred comments after hiding them, they'll disappear!)

Details

qemu-system-x86_64_G9JKLLDJq0.mp4

• Child comment threads are indented too much

Details

• Using the "navigate to post" feature after creating a post doesn't work

Details

qemu-system-x86_64_maqiiEIgTn.mp4

• Jumping to comment context (e.g., from notifications) shows the old page
• Threads with many comments show a spinner at the bottom (maybe paging isn't working?)

Details

qemu-system-x86_64_pwmuuKTnWE.mp4

• There is a weird gap in the cross-posts widget

Details

qemu-system-x86_64_YRDToj3w5V.mp4

• There is no pull-down to refresh (this isn't a big deal, and if the pulling down is either contradicting with another gesture or causing lots of rebuilds, maybe we don't need it)
  • Actually it looks like the refresh button doesn't work either
• The scrolling fixes from 1252 and 1319 seem to be missing. It should be possible to scroll the last comment to the top of the screen.
• When you create a comment, it does get displayed immediately, but it's not highligted.

On a bright note, I believe the following issues have been resolved (either from #1363 or #1361).

• Swipe anywhere to go back works much more consistently (Swipe goes back when actions are disabled #198)
• I haven't seen the random stretching issue (Interface stretching #461)

[hjiangsu]

The scrolling fixes from #1252 and #1319 seem to be missing

I was planning to add this back in in conjunction with the comment navigation!

[micahmo]

The scrolling fixes from #1252 and #1319 seem to be missing

I was planning to add this back in in conjunction with the comment navigation!

Makes sense! Just as long as you can scroll the last comment to the top even without using the nav buttons. 😊😊