-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Use cards for solutions page #3082
Conversation
I personally love 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.
I like it
content/solutions/index.html.haml
Outdated
min-height: 3rem; | ||
} | ||
.card img { | ||
height:96px; |
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.
Would be nice to have spacing consistent for css normally one space after the :. Harder for IDE and linter because it’s embedded in Haml I assume
It is definitely better than the original version 👍
Thanks!
…On Sun, Apr 19, 2020, 23:16 Tim Jacomb ***@***.***> wrote:
***@***.**** approved this pull request.
I like it
------------------------------
In content/solutions/index.html.haml
<#3082 (comment)>
:
> ---
-%ul
- - site.solutions.keys.sort.each do |key|
+:css
+ .card h3 {
+ font-size: 1.3rem;
+ min-height: 3rem;
+ }
+ .card img {
+ height:96px;
Would be nice to have spacing consistent for css normally one space after
the :. Harder for IDE and linter because it’s embedded in Haml I assume
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3082 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAW4RICQQOQP4MCIQZ4H5F3RNNS3NANCNFSM4ML5M7GA>
.
|
Looks really good :) |
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.
, thanks @zbynek !
Use cards for https://jenkins.io/solutions/ (reusing some styles from https://jenkins.io/participate/ and icons from https://simpleicons.org/ )
@oleg-nenashev WDYT?