unfolddata / ogdd

Open Data Maturity Dashboard
https://www.data.govt.nz/open-data/dashboard/
Other
0 stars 2 forks source link

Harmonise styles with data.govt header template #23

Closed tmfrnz closed 6 years ago

tmfrnz commented 6 years ago

New data.govt header content (logo and back-link) are within content container, limited to a width of 940px. Ideally dashboard app content should also be limited to same width.

Header

General

tmfrnz commented 6 years ago

@camfindlay we will mock up a few options for adjusting the style and will let you know what actions will be required on the CWP side (#root should possibly not be limited by data.govt container to at least allow the header bottom border to run all the way across)

tmfrnz commented 6 years ago

@camfindlay we have adjusted some design and styles to better match the data.govt template - you can re-deploy a WIP version from https://github.com/unfolddata/ogdd/tree/master/dist but please note that the application now requires the more specific root tag <div id="ogdd-root"></div> (#27)

With regards to the design and styles we suggest the following:

tmfrnz commented 6 years ago

You can also see the latest version on GitHub Pages: https://unfolddata.github.io/ogdd/

camfindlay commented 6 years ago

I see the dash specific logo has been retained -is this still to be removed?

camfindlay commented 6 years ago

Can confirm the svg graphics are working as expected.

camfindlay commented 6 years ago

I need to make some adjustments to styles on the minimal data.govt.nz header but overall fits much better.

tmfrnz commented 6 years ago

yes @camfindlay, if you do not mind we would like to retain the dash specific logo, but align the all the header wording

camfindlay commented 6 years ago

no worries, we'll retain the logo. tested on CWP, I think we can just about run with this release.

tmfrnz commented 6 years ago

Excellent @camfindlay - there are still a few minor issues we are working through (and also the about page content still needs updating) so please do not publish it before our go ahead... Thanks!

tmfrnz commented 6 years ago

@camfindlay did you by the way also white-list the 'Barlow' font? (see above for details)

camfindlay commented 6 years ago

font is fine. Let me know when you've fix minors at your end and I'll pick up here.

tmfrnz commented 6 years ago

@camfindlay just to let you know that we are pretty much final and addressed everything we planned to / we were aware of, so you can go ahead and grab the index.js and update your test page once more if you like. If you are happy and don't experience any further issues, the only changes should concern the data or the content (latter would require a re-build).

camfindlay commented 6 years ago

Other than updating the insights dataset, was there any thing else on your side that will require a rebuild of the index.js?

Otheriwse I'm going to roll a new install and we can deal with the repo handover at a later point (we'll deploy from our fork).

tmfrnz commented 6 years ago

There is nothing on our side that would require a rebuild but I am not sure if @enotsluap and his team have any other change requests (or have considered #30).

Of course the dashboard should not be made public until final go ahead from @enotsluap.

camfindlay commented 6 years ago

updates in draft - this issue can be closed.