Differential D16969

Wipe textarea immediately when submitting a new message in Conpherence
ClosedPublic
Actions

Authored by chad on Dec 1 2016, 6:06 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T11791: Lots of double posts in Conpherence

Commits

rP0db28cfc012d: Wipe textarea immediately when submitting a new message in Conpherence

Summary

Fixes T11791. We do this in durable column, but not in regular Conpherence. I think this is the right place? Not sure how this will feel with high lag.

Test Plan

Submit lots of text in a Conpherence.

Diff Detail

Repository

rP Phabricator

Branch

conph-double-post (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 14724
Build 19239: Run Core Tests
Build 19238: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 40838.Dec 1 2016, 6:06 PM

chad retitled this revision from to Wipe textarea immediately when submitting a new message in Conpherence.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T11791: Lots of double posts in Conpherence.

Herald added a subscriber: Korvin. · View Herald TranscriptDec 1 2016, 6:06 PM

epriestley accepted this revision.Dec 1 2016, 6:06 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 1 2016, 6:06 PM

Closed by commit rP0db28cfc012d: Wipe textarea immediately when submitting a new message in Conpherence (authored by chad, committed by chad). · Explain WhyDec 1 2016, 6:08 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

28 lines

webroot/

rsrc/

js/

application/

conpherence/

ConpherenceThreadManager.js

1 line

Diff 40838

View Options

resources/celerity/map.php