1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-25 06:50:55 +01:00
phorge-phorge/webroot/rsrc/js/application
epriestley 21f0aba701 Use an inline dialog element for inline comments in Differential
Summary:
The current approach of using a modal overlay dialog to create/edit inline
comments is pretty silly. Use an inline textarea instead.

This element isn't perfect and we have some mild modalness issues, but I think
it's better than the silly thing we've got going on right now. We can keep
poking it as people break it.

Test Plan:
  - Created comments; submitted and undid them in empty and nonempty states.
Used undo for nonempty states + cancel.
  - Edited comments; saved and canceled them. Used undo for changed state.
  - Replied to comments; yada yada as above.
  - Deleted comments.
  - Did various modal trickery where I clicked "Reply" on something else with a
dialog already up, this very mildly glitches but I think it's not a big issue.

Reviewers: vrana, btrahan, Makinde, nh

Reviewed By: vrana

CC: aran, epriestley

Maniphest Tasks: T431

Differential Revision: https://secure.phabricator.com/D1716
2012-02-29 14:28:48 -08:00
..
core Support paste file uploads 2012-02-23 16:36:58 -08:00
countdown Remove usage of JX.defer in favor of setTimeout 2011-08-10 18:09:59 -07:00
differential Use an inline dialog element for inline comments in Differential 2012-02-29 14:28:48 -08:00
diffusion Add preview and drafts to audits 2012-02-27 13:00:23 -08:00
herald Allow Herald to trigger audits for users or projects 2012-02-27 09:36:30 -08:00
maniphest Rough batch editor for Maniphest 2012-02-24 13:00:48 -08:00
owners Bring Javelin into Phabricator via git submodule, not copy-and-paste 2011-05-08 13:20:10 -07:00
phriction Add a document preview to Phriction 2011-07-16 18:48:27 -07:00
projects Remove "Former" project members 2011-12-16 17:46:02 -08:00
repository Tie all the pieces for symbol cross-references together 2011-10-09 17:58:17 -07:00
uiexample Examples using JX.View 2011-11-06 15:17:00 -08:00