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

Bezos is now worth $186 billion #48

Closed

Conversation

jobveldhuis
Copy link

The only change to this code is the change that was proposed in #45. Commits in repo are also partial translations to Dutch, but they have all been undone by a5e3ee2.

Translated the page meta and removed tracking to comply with European GDPR regulations
@jobveldhuis jobveldhuis changed the title Merge feature/update-bezos-wealth into master Bezos is now worth $186 billion Aug 7, 2020
Copy link
Owner

@MKorostoff MKorostoff left a comment

Choose a reason for hiding this comment

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

This just changes the label, without changing the size of the bar graph or running counter.

@jobveldhuis
Copy link
Author

I have done all the requested changes in 7be2a0a, both by updating the css to have the rule reflect the size of Bezos' newfound wealth and by updating the javascript, so the counter is displaying the correct amount of information throughout.

@MKorostoff
Copy link
Owner

Superseded by #51

@MKorostoff MKorostoff closed this Aug 27, 2020
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

2 participants