Page MenuHomePhabricator

Add back the motivator panel
ClosedPublic

Authored by chad on Feb 7 2017, 10:05 PM.
Tags
None
Referenced Files
F13148649: D17324.diff
Sat, May 4, 3:17 AM
Unknown Object (File)
Thu, May 2, 2:09 PM
Unknown Object (File)
Thu, May 2, 7:47 AM
Unknown Object (File)
Sat, Apr 27, 9:28 PM
Unknown Object (File)
Thu, Apr 25, 1:37 AM
Unknown Object (File)
Wed, Apr 24, 5:48 PM
Unknown Object (File)
Fri, Apr 19, 2:32 AM
Unknown Object (File)
Fri, Apr 19, 2:32 AM
Tokens
"Burninate" token, awarded by thoughtpolice."Love" token, awarded by cburroughs.

Details

Summary

Fixes T12226, Ref D17233. Resurrects the motivator panel.

Test Plan

Add panel, see fact on hover.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15560
Build 20522: Run Core Tests
Build 20521: arc lint + arc unit

Event Timeline

FWIW, this is the thing that I point someone at when I want to show them how to contribute changes to Phabricator, like just go through the workflow.

Through tireless research, I have discovered additional cat facts which may be candidates for inclusion:

Outside of the CAT scan, cats have made almost no contributions to modern medicine.
In ancient Egypt, the cat-god Horus watched over all cats.
The word "catastrophe" has no etymological relationship to the word "cat".
Many cats appear black in low light, suffering a -2 modifier to luck rolls.
The popular trivia game "World of Warcraft" features a race of cat people called the Khajiit.

This revision is now accepted and ready to land.Feb 8 2017, 5:01 PM

The popular trivia game "World of Warcraft" features a race of cat people called the Khajiit.

😹

This revision was automatically updated to reflect the committed changes.

We could make clicking the item (or any other item) pop up a dialog now fairly easily if you want -- not sure if that's the best direction to take the grand future of Cat Facts, but it's on the table as an easy technical thing if it makes sense here or elsewhere.