Page MenuHomePhabricator

Remove ajaxey comment magic in Pholio, and also some bugs
ClosedPublic

Authored by epriestley on Jun 21 2014, 6:03 PM.
Tags
None
Referenced Files
F18942600: D9649.diff
Tue, Nov 11, 12:22 PM
F18828244: D9649.id23158.diff
Fri, Oct 24, 3:18 PM
F18816129: D9649.id.diff
Tue, Oct 21, 5:06 AM
F18788841: D9649.id.diff
Oct 15 2025, 6:45 AM
F18788451: D9649.diff
Oct 15 2025, 5:00 AM
F18412582: D9649.id.diff
Aug 30 2025, 6:47 AM
F18387546: D9649.diff
Aug 29 2025, 3:27 AM
F18083644: D9649.id23161.diff
Aug 5 2025, 10:57 AM
Subscribers

Details

Summary

Fixes T5424.

  • One concrete issue: drafts were not being cleared properly because __draft__ was not set on submission. This (mostly) fixes phantom drafts.
  • This ajax comment magic feels weird and floaty and generally has problems. For example, if you add subscribers or inlines, all the stuff on the page which represents those won't update automatically. Instead, just reload. Maybe we'll ajax this stuff some day, but it feels like a net negative for now.
  • Also remove it from other applications where it's currently used.
  • Fix an issue with inline previews.
Test Plan

Made some comments on a mock, everything worked normally like I expected it to.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Remove ajaxey comment magic in Pholio, and also some bugs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 21 2014, 7:17 PM
epriestley updated this revision to Diff 23161.

Closed by commit rP9f2efd0fa048 (authored by @epriestley).