first draft tooling update

cmu-delphi / www-main

Delphi's homepage.

https://delphi.cmu.edu/

MIT License

5 stars 6 forks source link

first draft tooling update #964

Closed dsweber2 closed 1 month ago

dsweber2 commented 2 months ago

The tooling is woefully out of date. This is an attempt to fix that. It updates the drop-down menu, and the overview page.

I also added some css to render sub-menu's differently, since the list was rather long.

fix #864

netlify[bot] commented 2 months ago

Deploy Preview for cmu-delphi-main ready!

Name	Link
Latest commit	d4001c28e7f55ff33df00f3a6563cc78b1290b6e
Latest deploy log	https://app.netlify.com/sites/cmu-delphi-main/deploys/663407f09d538d00088b0ae2
Deploy Preview	https://deploy-preview-964--cmu-delphi-main.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

dshemetov commented 2 months ago

Thanks for diving in! Getting this visual bug, maybe we can avoid it by just calling that entry "Dashboards"?

dsweber2 commented 2 months ago

If you mean the new floating over the edge, I think that's an element to highlight what's changed

But yeah, might as well just call it dashboards.

Oh, and to be clear, @brookslogan, @nmdefries, @dshemetov, if you end up making larger changes/suggestions, feel free to just push directly to this branch, it'd probably be easier/more convenient than PR/PR's.

dsweber2 commented 2 months ago

updated based on live feedback during meeting. Could probably use some help on wording and making forecasting wins a bit clearer

dshemetov commented 1 month ago

Updated a few things:

renamed new Packages section to Code, did an editing pass on the language around that
removed a few unused shortcodes
changed the Our API link on the landing page to be Our Tools and linked to the Tools Overview instead of the very dense and outdated https://cmu-delphi.github.io/delphi-epidata/
updated the footer to contain a link to Code

nmdefries commented 3 weeks ago

Comments from Roni

The Delphi Epidata Dashboard in the pulldown menu is fine, but the Delphi Epidata Dashboard link within the "Dashboards" pulldown item gets a 404.

Also, in the Tools pulldown menu, "EpiVis" should be replaced with the more descriptive "Epi Signal Visualization".

dshemetov commented 3 weeks ago

@nmdefries Thanks! I'll make a fix in another branch.

cmu-delphi / www-main

first draft tooling update #964

✅ Deploy Preview for cmu-delphi-main ready!

Deploy Preview for cmu-delphi-main ready!