Page MenuHomePhabricator

Add back the motivator panel
ClosedPublic

Authored by chad on Feb 7 2017, 10:05 PM.
Tags
None
Referenced Files
F14483166: D17324.id41666.diff
Mon, Dec 30, 3:03 PM
Unknown Object (File)
Tue, Dec 24, 5:34 AM
Unknown Object (File)
Mon, Dec 9, 12:26 PM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Mon, Dec 9, 9:29 AM
Unknown Object (File)
Mon, Dec 9, 9:27 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.