Page MenuHomePhabricator

Only show loading animation on thread change in Conpherence
ClosedPublic

Authored by chad on Oct 14 2016, 2:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 25 2026, 8:55 PM
Unknown Object (File)
Mar 10 2026, 11:51 AM
Unknown Object (File)
Mar 4 2026, 12:47 AM
Unknown Object (File)
Feb 26 2026, 8:28 PM
Unknown Object (File)
Jan 30 2026, 3:33 AM
Unknown Object (File)
Jan 10 2026, 4:01 PM
Unknown Object (File)
Dec 23 2025, 4:35 AM
Unknown Object (File)
Dec 22 2025, 1:19 PM
Subscribers

Details

Summary

Fixes the send on enter flash, only uses the Threads loading animation on changing threads, not sending a message.

Test Plan

Change threads, post a message.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Only show loading animation on thread change in Conpherence.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

in theory you should do like this:

// this first
function thing() {
}

// this later
thing();

You can also trick the linter like this:

thing();

var thing = function() {
  // sneaky!
};

In practice this does not matter too much but sometimes the code is a little cleaner?

This revision is now accepted and ready to land.Oct 14 2016, 2:53 AM
This revision was automatically updated to reflect the committed changes.