[ENH] create lists for language, input type, data type for converters tables

[Remi-Gau]

closes #248

[netlify]

✅ Deploy Preview for astounding-beignet-8ca2d6 ready!

Name	Link
🔨 Latest commit	904f9bc
🔍 Latest deploy log	https://app.netlify.com/sites/astounding-beignet-8ca2d6/deploys/6396ced4c07276000db0c712
😎 Deploy Preview	https://deploy-preview-261--astounding-beignet-8ca2d6.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 settings.

[sappelhoff]

it's funny: clicking on the netlify deploy preview works pretty well --> UNTIL you click on the "benefits" tab ... which redirects you from the deploy PREVIEW to the actual deployment 😕 can you replicate this @Remi-Gau?

[Remi-Gau]

it's funny: clicking on the netlify deploy preview works pretty well --> UNTIL you click on the "benefits" tab ... which redirects you from the deploy PREVIEW to the actual deployment confused can you replicate this @Remi-Gau?

Yup I had seen that before even locally sometimes (I think).

[Remi-Gau]

fyi the actual page would be this one:

https://deploy-preview-261--astounding-beignet-8ca2d6.netlify.app/benefits.html#mri-and-pet-converters

[Remi-Gau]

seems to only affect the dropdown menu of the navbar

not sure why

[effigies]

This table is extremely wide. On my laptop, the left/right borders are off the edge of my scrollable screen. On a large desktop monitor I can see everything.

I think we need something more compact.

[Remi-Gau]

agreed this has been bugging me for a while actually

I have started playing with the CSS but not found something that would make it more manageable / readable if we keep some much info into the table.

[Remi-Gau]

probably need to do the same about the BEP table

[effigies]

I think there might be a solution with colored tags/badges and a legend. Modality, language, GUI, docs, license could all be represented very compactly.

[Remi-Gau]

@effigies

https://deploy-preview-261--astounding-beignet-8ca2d6.netlify.app/benefits.html#mri-and-pet-converters

how does that look to you?

[effigies]

Definitely more reasonably sized. I would also change the pypi and docker links to point to the projects on pypi and docker. We could also turn them into shields. Maybe it would make sense to just directly put the shield for the project into the YAML rather than rendering it on the fly with Jinja.

[Remi-Gau]

Definitely more reasonably sized. I would also change the pypi and docker links to point to the projects on pypi and docker. We could also turn them into shields.

Will look into that.

Maybe it would make sense to just directly put the shield for the project into the YAML rather than rendering it on the fly with Jinja.

Not sure. Given that we use the repos url for other things (want to see if we can "easily" generate a page for each tool where all the info for this tool is displayed). So would not make that change just yet.

[Remi-Gau]

@effigies @sappelhoff

now with dockerhub, pypi, conda-forge badges

https://deploy-preview-261--astounding-beignet-8ca2d6.netlify.app/benefits.html#mri-and-pet-converters

[sappelhoff]

nice! However, the conda badge doesn't say "conda", but when that conda package was "last updated", which is not the important information that users would need:

[Remi-Gau]

nice! However, the conda badge doesn't say "conda", but when that conda package was "last updated", which is not the important information that users would need:

ouch I missed that bit.

will try to see if I can tweak that.

[Remi-Gau]

nice! However, the conda badge doesn't say "conda", but when that conda package was "last updated", which is not the important information that users would need:

The badge should now display the version on anaconda.org.