facuxpalacio / divan

Shiny app for exploratory functional DIVersity ANalysis
MIT License
0 stars 0 forks source link

Layout #5

Open facuxpalacio opened 3 years ago

facuxpalacio commented 3 years ago

Hey!

I was wondering if it would be better to use a dashboard instead of tabsets. This would occupy a lateral section of the page, but the general layout is much more aesthetic.

See how dashboards look:

https://rstudio.github.io/shinydashboard/

Have a good week!

Facu

emmajhudgins commented 3 years ago

I think it definitely has a better UX/UI - is there a big overhead in converting the code to be usable in shinydashboard? If not I think we should go for it.

facuxpalacio commented 3 years ago

Thanks Emma. From what I've seen, it seems not be a big headache. Let's see how it results...

Have a good weekend!

facuxpalacio commented 3 years ago

I finally moved our script to shinydashboard. It looks just amazing! I thought it was gonna be harder, but it was quite simple indeed, as the hierarchy of functions is very similar in both layouts.

I turned off your report section just to be sure that the main body worked. I'll continue tomorrow, there's still a lot of work to do.

Facu

emmajhudgins commented 3 years ago

It looks SO GOOD! It's noticeably slower to load on my computer, but hopefully won't be too slow on their servers.

And agreed there's lots to do. I've set aside time this week to finish my portions - sorry for the delay!

facuxpalacio commented 3 years ago

Yeaahhh, I'm so in love with this haha.

After moving the script to shinydashboards, I noticed that the chunk on the report threw the following error:

"Error in tagAssert(header, type = "header", class = "main-header") : Expected tag to be of type header"

It seems to be a minor issue, but haven't delved into this. Could you take a look? Remember there's no rush Emma.