Homec4science

Conpherence - make the files widget purdy-ish

Authored by Bob Trahan <bob.trahan@gmail.com> on Mar 16 2013, 07:41.

Description

Conpherence - make the files widget purdy-ish

Summary:
does the title and also a few other small tweaks

  • kills the init js behavior; now its part of menu where it belongs.
  • adds the underline to the icon when its toggled in the widget menu
  • fixed JS initialization errors on the "create conpherence" page. Note I still like keeping all that init stuff in one function because its typing the same data a bunch to be passed over to the JS layer. Other ways to accomplish this obvi...

Only fun wrinkle here is I think Chad intended me to display "when the file was attached". Instead, I display when the file was *uploaded*. I think this jives better with our version where you can't delete and all that. Files are pretty powerful, long-living objects in Phabricator land.

Test Plan: added files to a conpherence and noted widget loaded updated okay. added a file with no author (generated by the system) and verified it still rendered okay. switched between conpherences and verified proper data in each files widget. uploaded image and text files to check the icons were correct.

Reviewers: epriestley, chad

Reviewed By: epriestley

CC: aran, Korvin

Maniphest Tasks: T2530

Differential Revision: https://secure.phabricator.com/D5337

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Mar 16 2013, 07:41
Pushed
aubortJan 31 2017, 17:16
Parents
rPH0bd135b1e6dd: Tweak panel-filter shadow.
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPHa161617d8eb4: Conpherence - make the files widget purdy-ish (authored by Bob Trahan <bob.trahan@gmail.com>).Mar 16 2013, 07:41