A small change in the cri-projects resources presentation

CyberCRI / learn-ext

WeLearn Browser Extension

https://welearn.cri-paris.org

MIT License

11 stars 1 forks source link

A small change in the cri-projects resources presentation #167

Closed prashnts closed 4 years ago

prashnts commented 4 years ago

This addresses #144

I'm not super happy with the results though, so let's discuss it using preview site.

It works ok for cards that have an image, but not for without-image cases. I can either adjust the header size to accomodate the logo, or replace the entire bottom portion with a clearer message that its a cri project.

Screenshots of current version:

netlify[bot] commented 4 years ago

Deploy preview for welearn-dev ready!

Built with commit 753660296b3bd4d7c623b4e9a62e1ef1adbe360b

https://deploy-preview-167--welearn-dev.netlify.app

prashnts commented 4 years ago

An alternative could be this to replace the favicon and url from bottom and just highlight that its a project.

prashnts commented 4 years ago

Ok, this works. We can make this release tomorrow. :)