-
Notifications
You must be signed in to change notification settings - Fork 131
Project Github tracker #117
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
base: main
Are you sure you want to change the base?
Conversation
|
I have these things left which will be done today (monday 28/2)
|
…comments, made token secret
I have some major issues with the chart and number of commits. I will ask in StackOverflow for some help with this. |
…ith position absolute
|
|
||
|
|
||
| /* TABLET */ | ||
| @media screen and (min-width: 667px) and (max-width: 1024px) { |
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.
I think it's enough with the min-width in tablet mode, cause your desktop starts with the min-width 1024 anyway, just a thought
NabeelMansour
left a comment
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.
Overall excellent job Mathilda and specially the accordian effects on your repoes
|
|
||
|
|
||
| /* TABLET */ | ||
| @media screen and (min-width: 667px) and (max-width: 1024px) { |
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.
I think it's well suited with just adding the min-width: 667px and skip the 1024 in tablet mode, cause of in the desktop it starts with the 1024 anyway as you can see the desktop mode, just a thought.
No description provided.