Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set minimum width #777

Open
matthew-white opened this issue Apr 24, 2023 · 2 comments
Open

Set minimum width #777

matthew-white opened this issue Apr 24, 2023 · 2 comments
Labels
frontend Requires a change to the UI styles Changes to CSS

Comments

@matthew-white
Copy link
Member

We know that Frontend doesn't work particularly well on mobile (#271). We've decided not to prioritize the mobile experience, but we could prevent unexpected behavior on mobile by setting a minimum width for Frontend. That would prevent overlapping elements or content overflow, for example. By setting a minimum width, the user would need to scroll horizontally on mobile to see content on the righthand side of the page.

@issa-tseng designs for a width of 1000px, so we could use that for the minimum width.

@lognaturel says that even today on mobile, things don't generally break. Typically the user is looking for something on the lefthand side of the page. However, we think that setting a minimum width would be an improvement.

After we set a minimum width, we will have bounds on the width on both ends: getodk/central-frontend#447 was about setting a maximum width. (The minimum width will likely be set on a different element from the maximum width, but this is just to say that the width of most Frontend content will end up becoming fully bounded.)

@matthew-white
Copy link
Member Author

Right now, there shouldn't ever be a horizontal scrollbar on the page as a whole. However, in getodk/central-frontend#894, I observed that when the entity conflict summary table had a horizontal scrollbar, the page also had a scrollbar. I'm really not sure why that is. I didn't see any overflow from the conflict summary table or the conflict summary box that would cause the page to need a scrollbar. I tried a few things, but I wasn't able to prevent the scrollbar from appearing, so I hid it entirely. However, we will need to show it again once we address this issue. At that time, I think we should take another look at the conflict summary table and try to figure out why there was a scrollbar on the page.

@matthew-white matthew-white transferred this issue from getodk/central-frontend Nov 9, 2024
@matthew-white matthew-white added the frontend Requires a change to the UI label Nov 9, 2024
@issa-tseng
Copy link
Member

one thing to think about here is to also set the zoom to a sane number if the mobile presentation is lacking

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Requires a change to the UI styles Changes to CSS
Projects
Status: 🕒 backlog
Development

No branches or pull requests

2 participants