Page MenuHomePhabricator

Ajax commenting on lightbox images
ClosedPublic

Authored by chad on Nov 21 2016, 9:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 10:32 AM
Unknown Object (File)
Sat, Apr 6, 9:02 AM
Unknown Object (File)
Sat, Apr 6, 9:02 AM
Unknown Object (File)
Fri, Apr 5, 6:05 AM
Unknown Object (File)
Fri, Apr 5, 3:32 AM
Unknown Object (File)
Thu, Apr 4, 12:56 PM
Unknown Object (File)
Thu, Apr 4, 2:52 AM
Unknown Object (File)
Thu, Apr 4, 2:45 AM

Details

Summary

Adds a comment box, you can put text into it, hit enter, and see it come back.

Test Plan

Put text into box, see it come back.

Diff Detail

Repository
rP Phabricator
Branch
file-comment (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14603
Build 19063: Run Core Tests
Build 19062: arc lint + arc unit

Event Timeline

chad retitled this revision from to Ajax commenting on lightbox images..
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.

I seem to recall you wanting to do this all in EditEngine, but you'd have to do that part?

chad retitled this revision from Ajax commenting on lightbox images. to Ajax commenting on lightbox images.Nov 21 2016, 10:28 PM
src/applications/files/controller/PhabricatorFileLightboxController.php
40

I'll clean this up if I get to land this.

epriestley edited edge metadata.

Yeah, this is fine. Seems real bizarre to me to put chatbox semantics in a lightbox, but whatevs. I'll consolidate this eventually.

This revision is now accepted and ready to land.Nov 22 2016, 12:20 AM

SoE? I can add a submit button.

We can try it, maybe it won't feel weird in practice. But I wouldn't expect send-on-enter when commenting on files normally.

I just tossed this all together to get it working, hadn't really put a lot of thought into it. copy-pasta'd the easiest thing.

chad edited edge metadata.
  • make wider
  • use profile photos
  • add submit button
This revision was automatically updated to reflect the committed changes.
In D16907#199999, @chad wrote:

SoE? I can add a submit button.

This comment was removed by epriestley.