-
Notifications
You must be signed in to change notification settings - Fork 54
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
docs: add ui kit and learn more to landing page #1556
Conversation
Deploy preview for fundamental ready! Built with commit befae70 |
929a3ce
to
3f8ddba
Compare
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.
The following comments are just a personal opinion as I am not a designer and can't really comment on this :)
- On very big screen the
Download
button looks weird. I think it should be centered. The 3 sections under it look weird- they are too small with a lot of space between them. The Learn more items are too big
- The spacing on the left and right side is not the same. We have more space on the left hand side which makes the items look like they are not centered.
- Overlapping of the "Go to top" arrow and content
- UI kit bundle sections look too close in Tablet view. Maybe show 2 sections per row in this case?
- Hamburger button showing when links are showing too in Tablet view
&__title { | ||
@extend %text; | ||
|
||
@include text(); |
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.
why did you change it to mixin?
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.
so that I could use it easily in media querys, it wasnt letting me without
3c19eae
to
848bbd0
Compare
Everything looks pretty good. I notice a few things for the large screen:
|
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.
Can you attach the files: UI Kit, etc...?
bd15483
to
befae70
Compare
Closes #1553
Add UI kit and learn more sections to landing page