Skip to content
This repository has been archived by the owner on Dec 8, 2017. It is now read-only.

Admin styles #73

Merged
merged 14 commits into from
Mar 17, 2016
Merged

Admin styles #73

merged 14 commits into from
Mar 17, 2016

Conversation

gemfarmer
Copy link
Contributor

What this PR does:

  • A first step towards transforming the look and feel of the Forecast Admin
  • Uses colors, styles, fonts that are consistent with the Forecast public facing application
  • Slight modification to default django admin

@stvnrlly Ready for review!

What still needs to be done:

  • Reconfigure django admin templates to allow the main content view on the / of the admin to be used as an admin-wide navigation. The breadcrumb is currently a placeholder for this, but can be removed once this navigation is put into place. I didn't do this for the first iteration because it involved considerable restructuring. With the styles in place, it shouldn't be terribly difficult to keep the same look with a reconfigured navigation strategy

*EDIT: more on reconfiguration:

@stvnrlly: Sure. I left it a little vague, I guess. The reconfiguration I was referring was making the left column on the mock above an admin wide navigation panel. That currently isn’t possible because app_list, the object that generates it isn’t permitted on every admin view. According to @jacobian this involves extending AdminSite with a custom each_context() method.
Because of this layer of difficulty, I decided to work with the default functionality until I have more bandwidth or someone who understands django better than myself can help me put that in place.

@gemfarmer gemfarmer changed the title [WIP] Admin styles Admin styles Mar 16, 2016
@stvnrlly stvnrlly self-assigned this Mar 16, 2016
@stvnrlly
Copy link
Member

🎆

stvnrlly added a commit that referenced this pull request Mar 17, 2016
@stvnrlly stvnrlly merged commit 6782938 into develop Mar 17, 2016
@gemfarmer
Copy link
Contributor Author

💥 Thanks again @stvnrlly !

@stvnrlly stvnrlly deleted the styles-admin branch June 8, 2016 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants