Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Reorganization of the dashboard with small styling update. #1527

Merged
merged 2 commits into from
Sep 26, 2017

Conversation

petrillek
Copy link
Contributor

Hi Jeff,
firstly thank you for all your energy you put into this project.
We have recently switched to it and so far it is really great experience.

As time goes I've added multiple sites with db's to my VM and the dashboard start to look full and confusing. I've also realized that parts of the dashboard are used more often then others.

This pull request is only changing the dashboard template (provisioning/templates/dashboard.html.j2).

Provided changes (see attached comparison screenshots):

  • Make all tables responsive, to prevent overlapping of content even on desktop.
  • Add stripes to some tables.
  • Fixed logo position/size on small screen.
  • Cleaner header.
  • First row – Site and DB list's.
  • DB list is much simpler.
  • Second row – Technical info (DB Info, PHP info, Development tools).
  • Third row – Host file content.
  • All new styling is done only with addition Bootstrap classes.

Hope you will find this at least partially useful.

Petr

drupal-vm-dashboard-desktop

drupal-vm-dashboard-small-screen

@geerlingguy
Copy link
Owner

A cursory glance looks good to me. I'd like to run it myself once just to be sure everything functions the same, but assuming it does, I'll merge into next minor release.

@petrillek
Copy link
Contributor Author

petrillek commented Aug 29, 2017

I just realize that it is probably better to move the condition for Database List label (if its only a list or if its support Adminer) between the macros leaving the actual template cleaner.

@geerlingguy geerlingguy merged commit a6426e5 into geerlingguy:master Sep 26, 2017
@geerlingguy
Copy link
Owner

Looks good to me, thank you for your work on sprucing up the template!

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.

None yet

2 participants