Page MenuHomePhabricator

Make Differential inline events actually trigger comment preview refreshes
ClosedPublic

Authored by epriestley on Jan 19 2017, 7:54 PM.
Tags
None
Referenced Files
F19554319: D17229.id41440.diff
Thu, Jan 29, 8:28 PM
F18822546: D17229.id41440.diff
Oct 23 2025, 7:23 AM
F18810682: D17229.diff
Oct 19 2025, 10:49 PM
F18787402: D17229.diff
Oct 14 2025, 11:35 PM
F18783888: D17229.diff
Oct 13 2025, 10:28 AM
F18745498: D17229.id41439.diff
Oct 3 2025, 7:26 AM
F18650409: D17229.id41440.diff
Sep 21 2025, 12:39 AM
F18612519: D17229.diff
Sep 14 2025, 9:10 AM
Subscribers
None
Tokens
"100" token, awarded by jmeador.

Details

Summary

Earlier, I made some changes so that when you create or edit an inline, the comment at the bottom of the page updates (even though you didn't fiddle with the stacked actions inputs).

At the last second I broke them by spelling this wrong while cleaning things up, so they didn't actually work. Spell the property correctly ("showPreview", not "shouldPreview").

Also, we have some JS which rewrites "Not Visible" into "View", but it fires in an inconvenient way now and is flickery for me. Ideally this should get cleaned up slightly better eventualy, but at least make is stop doing so much flickery layout for now.

Test Plan
  • Wrote no comment on a revision.
  • Added an inline.
  • Saw comment preview properly update immediately.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

webroot/rsrc/js/application/transactions/behavior-comment-actions.js
217

Here's the property being used elsewhere. My incorrect spelling, shouldPreview, appears nowhere in the codebase.

This revision is now accepted and ready to land.Jan 19 2017, 8:28 PM
This revision was automatically updated to reflect the committed changes.