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
F13103730: D17229.diff
Sat, Apr 27, 11:21 AM
Unknown Object (File)
Tue, Apr 23, 4:54 PM
Unknown Object (File)
Sat, Apr 20, 11:31 AM
Unknown Object (File)
Fri, Apr 19, 6:08 PM
Unknown Object (File)
Sat, Apr 6, 8:49 PM
Unknown Object (File)
Mon, Apr 1, 11:03 AM
Unknown Object (File)
Sun, Mar 31, 1:28 PM
Unknown Object (File)
Mar 27 2024, 6:59 PM
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.