search

OpenZeppelin / docs.openzeppelin.com

Source for the OpenZeppelin documentation site
https://docs.openzeppelin.com
45 stars 87 forks source link

🔧 Replace resource cards with bullet points and remove duplicated #393

Closed cairoeth closed 10 months ago

cairoeth commented 10 months ago

Fixes view of cards.

ernestognw commented 10 months ago

The UI issue is also happening in the main Learn section. For some reason I can't replicate in local but we should fix both more generally.

cairoeth commented 10 months ago

The UI issue is also happening in the main Learn section. For some reason I can't replicate in local but we should fix both more generally.

@ernestognw fix in the sense of replacing to bullet list or fixing the card style? tbh even if we fix the card style not sure it makes sense to use them in these cases (i think they are good for 3-4 cards maximum, otherwise too crowded).

ernestognw commented 10 months ago

@ernestognw fix in the sense of replacing to bullet list or fixing the card style? tbh even if we fix the card style not sure it makes sense to use them in these cases (i think they are good for 3-4 cards maximum, otherwise too crowded).

Right, after reviewing I think the change makes sense on its own. Still we need to submit another PR for fixing the cards issue.