Page MenuHomePhabricator

Fix some quirkiness with Answer comments in Ponder
ClosedPublic

Authored by chad on Aug 20 2015, 5:32 PM.
Tags
None
Referenced Files
F18112887: D13946.diff
Tue, Aug 12, 9:45 PM
F18085103: D13946.id.diff
Tue, Aug 5, 7:00 PM
F18082256: D13946.id33666.diff
Tue, Aug 5, 4:50 AM
F17932615: D13946.diff
Wed, Jul 30, 8:28 PM
F17855522: D13946.id33663.diff
Sun, Jul 27, 4:18 PM
F17729149: D13946.diff
Sat, Jul 19, 6:05 PM
F17632482: D13946.id.diff
Jul 10 2025, 3:23 PM
F17627537: D13946.id33663.diff
Jul 10 2025, 6:28 AM
Subscribers

Details

Summary

There is still some general buginess with answer comments, trying to work them out. This replaces timeline rendering into one offs (less performant) but resolves many bugs. Or if there is a more performant way, let me know? Also when leaving an answer comment, you currently get redirected back to the page, but both the comment form is still populated and you dont see your answer without a reload. I feel like I'm missing some magical parameter to pass, so just redirecting back to the question itself.

Test Plan

Leave lots of answer comments.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Fix some quirkiness with Answer comments in Ponder.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

I think this is fine, the performance cost shouldn't be substantial.

Not sure about the reload thing, I can take a look at some point.

This revision is now accepted and ready to land.Aug 20 2015, 6:13 PM
This revision was automatically updated to reflect the committed changes.