-
Notifications
You must be signed in to change notification settings - Fork 8
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
Press Page #23
Comments
cc @TomerFi about pagination in 11ty https://www.11ty.dev/docs/pagination/ here's an example of how we generate pages from data in red hat design system: https://github.com/RedHat-UX/red-hat-design-system/blob/694fd26b63fc1a49992c24a546b26167dff83ec3/docs/components/demos.html the linked example filters the items in the for this case, my thought was to generate an index page with let's say 10 items, and show the opengraph image and the text summary, with a "read more" link |
#31 did most of the work |
I recommend using the card component (with an image) for this page and for the "Posts" page to make it easier to read and to make it more visual and less textual/list view. Card UI: For cards without an image: let's try to find a relevant image for all of our posts (please make sure to use images with the right license, for example from this website). |
Thanks @ShiranHi this is great. |
This is great, @ShiranHi , thanks We'll need to figure out what to do when a card has multiple links, e.g. https://github.com/RedHat-Israel/red-hat-israel-site/blob/1dbecf95a054311f442c387190954b2bfdb13659/_data/publicity.yaml#L60-L76 This currently renders like so: <h2>גאווה ישראלית: יחידת ממר"ם זכתה בפרס החדשנות הטכנולוגית לשנת 2022 באירוע ה-Summit הגלובלי של רד האט</h2>
<p>קראו עוד באתרי</p>
<ul>
<li>
<a href="https://itnews.co.il/news/events-expos/?p=38337" rel="noreferrer nofollow" target="_blank">IT News</a>
</li>
<li>
<a href="http://www.talniri.co.il/marketnews/article.asp?id=108191" rel="noreferrer nofollow" target="_blank">Talniri</a>
</li>
<li>
<a href="http://www.softnews.co.il/news/fgbhj-jgegf.html" rel="noreferrer nofollow" target="_blank">Soft News</a>
</li>
<li>
<a href="https://www.funder.co.il/article/134309–" rel="noreferrer nofollow" target="_blank">Funder</a>
</li>
<li>
<a href="https://www.biti.co.il/mamram-red-hat/" rel="noreferrer nofollow" target="_blank">Biti</a>
</li>
<li>
<a href="https://www.ice.co.il/digital-140/news/article/860081" rel="noreferrer nofollow" target="_blank">ICE</a>
</li>
<li>
<a href="https://www.pc.co.il/upcoming-conferences/363732/" rel="noreferrer nofollow" target="_blank">אנשים ומחשבים</a>
</li>
</ul> current design specs don't say much about single-cards-with-multiple-links. cc @coreyvickery |
@bennypowers can these links be clickable labels/tags? |
not sure, better to ask @coreyvickery or @marionnegp (PTO until wed) |
@ShiranHi, our labels are typically used to show meta data, like categories for sorting info, so I'm not sure this would be the best application of labels. A group of Brick CTAs might work, depending on how wide the card will be. Otherwise, I've mostly seen multiple links in a card be used like this on the homepage or this on the Cloud Computing page. |
@ShiranHi It is not wise to show too many links in a card, especially if there are multiple cards in one row; a user might have a hard time making a choice if there are too many things to select. I like how your layout above has only one CTA or link per card. You may need to use a wider card or band layout if you wish to include more than two CTAs or links in one space. Let us know if you need further design guidance. |
I agree @coreyvickery , experience wise it is better to have only one CTA. |
@ShiranHi I know you asked Benny, but that sounds like a nice solution to me. |
The text was updated successfully, but these errors were encountered: