MuckRock / documentcloud-frontend

DocumentCloud's front end source code - Please report bugs, issues and feature requests to info@documentcloud.org
https://www.documentcloud.org
GNU Affero General Public License v3.0
15 stars 5 forks source link

Updates application `MainLayout` #504

Closed allanlasser closed 6 months ago

allanlasser commented 6 months ago

I updated the application's main layout in response to lessons learned:

I also made some data fetching changes:

I also fixed some styles.

(I'm not satisfied with the mobile header as-is, but the point in this PR is that the slotted sidebars are visible as drawers from there. If we want to move around user and org menus, we can do that later on.)

Screenshots

Components Main Layout Desktop

Components Main Layout Desktop (1)

netlify[bot] commented 6 months ago

Deploy Preview for documentcloud-frontend ready!

Name Link
Latest commit 8de1d3415eaf7760bfd3e624b33388ac6b43221f
Latest deploy log https://app.netlify.com/sites/documentcloud-frontend/deploys/660ed59a3c741800090ff3ea
Deploy Preview https://deploy-preview-504.muckcloud.com
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

eyeseast commented 6 months ago

image

Getting an overflow somewhere. Might be in the top nav. This is on my iPhone XR.