Homec4science

Allow any EditEngine comment form to be pinned

Authored by epriestley <git@epriestley.com> on Dec 28 2016, 18:39.

Description

Allow any EditEngine comment form to be pinned

Summary:
Fixes T12049. This expands "Haunted" comment panels to EditEngine, and by extension to all EditEngine applications.

Eventual goal is to remove custom commenting code in Differential and replace it with EditEngine code.

Changes from current "haunt" mode:

  • This only has one mode ("pinned"), not two ("pinned", "pinned with preview"). There's an inline preview and scroll behavior is a little better.
  • Now has a UI action button.

Slightly tricky stuff:

  • This interacts with "Fullscreen" mode since it doesn't make sense to pin a full-screen comment area.
  • This should only be available for comments, not for remarkup fields like "Description" in "Edit Task".

Test Plan:

  • Pinned/unpinned in Maniphest.
  • Pinned/fullscreened/unfullscreened/unpinned.
  • Checked that "Edit Task" doesn't allow pinning for "Description", etc.
  • Pressed "?", read about pressing "Z".
  • Pressed "Z".

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12049

Differential Revision: https://secure.phabricator.com/D17105

Details

Committed
epriestley <git@epriestley.com>Dec 29 2016, 21:49
Pushed
aubortJan 31 2017, 17:16
Parents
rPH4d8ac0060235: Add missing "array" typehint to DifferentialCommitMessageParser
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH3fedc8c299f7: Allow any EditEngine comment form to be pinned (authored by epriestley <git@epriestley.com>).Dec 29 2016, 21:49