Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gobierto Indicators module #248

Merged
merged 13 commits into from
Jan 20, 2017
Merged

Gobierto Indicators module #248

merged 13 commits into from
Jan 20, 2017

Conversation

ferblape
Copy link
Member

This PR implements the sandbox of #65

@martgnz says this is OK for merging as a v1.

Works

  • General card implementation with percentage changes, easy to customize
  • Three main graphics. One static and the two others with nice voronoi-powered tooltips.

To be done

  • Implement fixed header on scroll (with subsections ?)
  • Add different card types
    • Sparkline background card
  • Improve the existing graphics
    • Age distribution: add population projections? add comparison between province, state and genders.
    • Unemployment: comparison with province, probably all the cities (w/ light lines). Comparison with state.
    • Rent: button to switch between gross and net rent

Needs review

  • Check if the indicators are correct (percentages, currency where is needed).
  • Data review: there is a lot of data transformation going on in the background for some graphics. We should check if our math and code is correct.

cc @ferblape @furilo.

Sections skeleton

Minimal header and intro

Better header structure

Better intro

Add basic widget structure and style

Improve header classes

Push all the sections

Refactor nav a bit, prepare for fixed position

Upload png

CSS refactor

Move the utils

Fix active section decoration

Responsive navbar

Add links to share buttons, improve decoration

Widget info hover styles

Sandbox menu items

New design for the indicators page, start working on the graphics

Chart structure

Age distribution chart draft

Add chart headline, various fixes

Improve axis handling

Fix merge

Read TBI API token from the sandbox

Fix age distribution responsive handler

WIP unemployment calculation

WIP cards data retrieval implementation

Implement cards percentage variations

Isolate card indicators painting code

Populate the indicator cards and handle different data types

Indicators refactor. Flexible card data retrieval for and metadata WIP

Paint all the indicators

Fix the indicators

Fix rendering and age distribution vis x axis adjustment

Data driven titles, calculate city average age

Show dataset info on cards

Improve indicator cards interaction, add all the info

Prepare unemployment by age pct values

Another way to calculate the pct

Add lodash

Add scatterplot WIP

Fix responsive handler

Calculate ratio

Add Spanish locale

Reference D3 Spanish locale on application.js

Fix age distribution locale

Remove Spanish locale, as its global now

Scatterplot: Add custom formatter

Scatterplot text label of current city

SVG annotation

Switch to a Plasma color scale & fix the encoding

Implement voronoi and tooltip WIP

Unemployment line chart WIP

Add top margin

Fix extent and pct calculation for the last year

Change y axis alignment

Make the line chart responsive

Unemployment by sectors chart

Implement voronoi tooltip

Add distance limited voronoi tooltip

Age distribution styles

Improve responsive handler

Fix responsive handler

Fix scatterplot styles

Remove legacy annotation function and label

Improve custom formatter
@ferblape ferblape force-pushed the 65-indicators-sandbox-squashed branch from d59d5bf to 1c33bb7 Compare January 13, 2017 15:56
@ferblape ferblape changed the title Minimal header and intro Gobierto Indicators module Jan 13, 2017
@ferblape ferblape force-pushed the 65-indicators-sandbox-squashed branch from d48f940 to 1cfbcd8 Compare January 13, 2017 16:58
@codecov-io
Copy link

codecov-io commented Jan 13, 2017

Current coverage is 90.81% (diff: 100%)

Merging #248 into stable will decrease coverage by 0.02%

@@             stable       #248   diff @@
==========================================
  Files           181        181          
  Lines          3976       3976          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           3612       3611     -1   
- Misses          364        365     +1   
  Partials          0          0          

Powered by Codecov. Last update 90802f7...7a2f3f4

@ferblape
Copy link
Member Author

I have just released this branch to staging. We can see real indicators for Madrid in http:https://madrid.gobify.net/indicadores

Next steps: @furilo and @ferblape to review and create issues if necessary.

@ferblape ferblape mentioned this pull request Jan 20, 2017
3 tasks
@ferblape ferblape merged commit 5394f03 into stable Jan 20, 2017
@ferblape ferblape deleted the 65-indicators-sandbox-squashed branch January 20, 2017 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants