-
Notifications
You must be signed in to change notification settings - Fork 87
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
Project - Jason Brunner #80
Conversation
✅ Deploy Preview for mucss-dataviz ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
…I still want to try and fix the lengend).
Will double check for spelling errors in the next days. Open for any feedback
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Some comments below.
…would be nicer to not need to just copy past the ones for the labels down, but when I try to put them in to a variable i get an error. First i thought its caused by the geom_point parts but also just the ones with annotate dont work. I also tried bypassing it by formating them as a list, but this did not work aswell :/. I hope its still allright like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, merging!
Congrats, here you are: https://csslab.uc3m.es/dataviz/projects/2024/100543010/ |
My first draft.