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

Peer review semi-final assignment #1

Open
HappyPantss opened this issue Nov 26, 2020 · 0 comments
Open

Peer review semi-final assignment #1

HappyPantss opened this issue Nov 26, 2020 · 0 comments

Comments

@HappyPantss
Copy link

First impression
First of all, I have globally scanned your page, and it looks amazing!

Your title, to begin with, stands out! I like it how you change the subject to what your concept is about. Well, you don't change the subject of the assignment, but you show what it's really about.

Be aware to not post a lot of introduction text in the header, I think that you only have the title in there, suits the page very well. The idea of an introduction text is good, but maybe place it somewhere else.

Story first half:
I think you story is very well written. But I don't know the meaning of 'Autoluw'. Maybe you can explain this somewhere? Or if you've allready done that, make it clear where.

Viz first half
The visualisation you made here, is really nice! The only thing I don't get is why you don't used Rotterdam as a bar. Because you're telling us Rotterdam has the most P+R locations. So why don't show the difference?

Story second half:
Very clear!.. maybe you can write the tip "De onderstaande kaart is interactief: zoom.." a little bit closer to the map.

Viz second half:
Legend is clear! Maybe you can put the bubbles of the sizes in eachother!

The map is so well done! I like it very much. The way you can zoom, see all the sizes and lines of locations that belong to that same city is so well thought out.

General:
Overall, I think you nailed it. Your text is well written, visualizations are beautiful. Maybe you can make the padding around the text somewhat bigger, so its more readable.

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

No branches or pull requests

1 participant