refactor: simplify sidebarlayout and removed nested divs#1156
refactor: simplify sidebarlayout and removed nested divs#1156jderochervlk wants to merge 13 commits intomasterfrom
Conversation
Deploying rescript-lang-org with
|
| Latest commit: |
af23177
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://634efa5f.rescript-lang.pages.dev |
| Branch Preview URL: | https://vlk-fix-sidebarlayout-overfl.rescript-lang.pages.dev |
|
Hmmm, is the smaller left padding deliberate? |
Nope, I have a few things to cleanup. |
…nto vlk/fix-sidebarlayout-overflow
Fixed. |
|
Ah so you wanted to put the sidebar actually on the side now? |
Moving it to the side wasn't intended. I was simplifying some of the divs to address content overflowing at different break points. It shouldn't look too different from prod right now. |
…nto vlk/fix-sidebarlayout-overflow
|
Closing this out because it became too complicated. I have a better idea on how to tackle this. |





There were a lot of extra divs leading to some layout wonkiness at different screen sizes.
Before
After