cncf / cncf.io

☁️♮🏛🚧 The CNCF.io WordPress website
https://cncf.io
MIT License
80 stars 37 forks source link

Updated landscape block to use landscape 2 scripts #820

Closed cjyabraham closed 5 months ago

cjyabraham commented 5 months ago

As per #819

Pages with landscape 2 embedded:

Notes:

  1. One difference is that when you click on a tile in the new landscape, it pops open a new tab and opens the modal popup for the particular tile. The old landscape would just open the modal directly within the iframe. I'm not sure if this is a problem but just wanted to call it out.
  2. Since the tiles don't show any metadata, the two Juniper tiles on the CNF page appear to be the same when they're not. Maybe we can live with this?
  3. The Certified Kubernetes listing includes an extra category: "PaaS/Container Service (21)", which is not listed on the current landscape listing. Not sure if this is desired or not?
tegioz commented 5 months ago

This looks great @cjyabraham 🙂

Some comments to your notes:

  1. This was intentional to drive more traffic to the landscape site. These views are embedded in other sites in addition to the CNCF website and users may find interesting to discover the full landscape experience. But we can change how this works. Or maybe as an intermediate point, we could display some more information when hovering over an item (like we do on the landscape site, showing a small card) and keep the click behavior as it is.

  2. We'd like to add more styles for the tiles in addition to the 3 supported right now. We could add one with the items names, or even a card style that would look similar to the card view mode. Just let us know which one you'd prefer and we'll take care of it.

  3. At the moment it's possible to embed either a full category (with all its subcategories) or a single subcategory. We've done this in a generic way so that it can be used across all landscapes: there's nothing CNCF specific or pre-configured in any way, it's all dynamic. But we could try to make the content selection a bit more fine grained if needed and allow to exclude certain subcategories when displaying a full category.

cjyabraham commented 5 months ago

@caniszczyk do you have any thoughts on the above points? If good enough for now, should I go ahead and deploy?

@tegioz for (2), I could see having the titles added and also the full card view mode would both be useful embed formats.

tegioz commented 5 months ago

@tegioz for (2), I could see having the titles added and also the full card view mode would both be useful embed formats.

Cool, we'll get it done 👍

caniszczyk commented 5 months ago

Let's deploy and roll it out for a week or so to see what happens.

Once that is done, we can start deprecating the old landscape.

On Fri, Jan 5, 2024 at 8:06 AM Sergio Castaño Arteaga < @.***> wrote:

@tegioz https://github.com/tegioz for (2), I could see having the titles added and also the full card view mode would both be useful embed formats.

Cool, we'll get it done 👍

— Reply to this email directly, view it on GitHub https://github.com/cncf/cncf.io/pull/820#issuecomment-1878277026, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAPSINHPCAYFIKZO7PQOQ3YM6YBNAVCNFSM6AAAAABBL4MBI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGI3TOMBSGY . You are receiving this because you were mentioned.Message ID: <cncf/cncf. @.***>

-- Cheers,

Chris Aniszczyk https://aniszczyk.org

cjyabraham commented 5 months ago

Ok, these are live: