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
F15546178: D16705.diff
Sat, Apr 26, 2:12 PM
F15459655: D16705.id40222.diff
Mon, Mar 31, 2:57 PM
F15456850: D16705.diff
Sun, Mar 30, 12:38 PM
F15418541: D16705.id.diff
Mar 20 2025, 11:56 PM
F15411363: D16705.id40221.diff
Mar 19 2025, 9:13 AM
F15353087: D16705.diff
Mar 10 2025, 7:59 PM
Unknown Object (File)
Feb 23 2025, 7:06 AM
Unknown Object (File)
Feb 18 2025, 11:53 AM
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.