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

Recreate Data Bias interactive #1185

Merged
merged 20 commits into from
Nov 5, 2019
Merged

Recreate Data Bias interactive #1185

merged 20 commits into from
Nov 5, 2019

Conversation

courtneycb
Copy link
Contributor

@courtneycb courtneycb commented Nov 4, 2019

Relates to #1182

Changes

  • Rebuild interactive for our system
  • Licence appropriately
  • Add README

Added features

Below are some features I added/altered that were not in the original interactive:

  • Every time the interactive is loaded/restarted the background colour is randomly chosen.
  • I removed the 'count' of circles that were clicked as I wasn't sure what educational value this added to the interactive.
  • Slider background shows gradient of colours.
  • A border glow will appear around a circle once it is clicked (instead of turning the circle white).

Notes

  • I'm not sure what should happen in the case that a user does find all of the circles (it is simple to inspect the page and find all of the circle divs to click). I feel if a user does this they are purposely going out of their way to miss the point of the interactive anyway.

  • It is theoretically possible that all circles on load of the interactive are all the same colour, however the probability of this is very very small. I did not come across this once while building and testing the interactive.

  • It is theoretically possible that all of the 'hidden' circles may partly overlap other circles, revealing their position. However I did not encounter this at all while building and testing the interactive (some overlapped but never all).

This interactive is for the chapter 'Big Data' that doesn't exist yet.

@courtneycb courtneycb requested a review from eAlasdair November 4, 2019 21:16
@courtneycb courtneycb self-assigned this Nov 4, 2019
Copy link
Contributor

@eAlasdair eAlasdair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love the new layout, good work 🥇

Copy link
Contributor

@eAlasdair eAlasdair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, just suggestions I've already made here and here, plus one teeny tiny bug

@courtneycb courtneycb merged commit ac2741b into develop Nov 5, 2019
@courtneycb courtneycb deleted the issue/1182/data-bias branch November 5, 2019 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants