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
F13108396: D17229.diff
Sun, Apr 28, 1:56 AM
F13108063: D17229.diff
Sun, Apr 28, 12:14 AM
Unknown Object (File)
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
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
Branch
inlinex1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15328
Build 20179: Run Core Tests
Build 20178: arc lint + arc unit

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.