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

Handle tables with many columns #358

Open
alandefreitas opened this issue Oct 30, 2024 · 4 comments
Open

Handle tables with many columns #358

alandefreitas opened this issue Oct 30, 2024 · 4 comments
Assignees
Labels
HTML/CSS The issue pertains to the html templates or stylesheets.

Comments

@alandefreitas
Copy link
Member

Boost.URL has this table that's too large and ends up outside the area of the main content:

https://www.boost.org/doc/libs/master/doc/antora/url/reference.html

Image

There should be a solution where we can scroll right or some other option.

Of course, we can also rethink this individual Boost.URL table format. We migrated it from previous QuickBook documentation. But I expect this problem to reoccur with other libraries.

@rbbeeston
Copy link
Member

Tables are generally a pain to deal with in this format. My preferred method would be to allow the table to scroll inside the div, and not allow it to go outside the div, but there are other who are strongly opposed to that.

The alternatives would be to find ways to shrink the table to force it to conform to the size or allow the container div to expand, even if it goes outside of pre-set boundaries.

@rbbeeston rbbeeston added Bug A problem with the UI or stylesheets HTML/CSS The issue pertains to the html templates or stylesheets. and removed Bug A problem with the UI or stylesheets labels Oct 30, 2024
@rbbeeston rbbeeston moved this to Accepted in website-v2 Oct 30, 2024
@rbbeeston rbbeeston assigned cdw9 and unassigned julioest Nov 12, 2024
@cdw9
Copy link
Collaborator

cdw9 commented Nov 18, 2024

I think it makes sense to have a horizontal scroll when necessary - it is already being done on the code blocks. It's also going to be needed on mobile devices.

Ultimately, the tables should be reviewed to see if the data actually needs to be in a table. In this case, it could be presented in a different way.

@cdw9
Copy link
Collaborator

cdw9 commented Nov 18, 2024

I committed a fix, it's currently part of https://github.com/boostorg/boostlook/pull/43/commits

@rbbeeston
Copy link
Member

One of the downsides to the way the data is handled is that we don't have much control over what we get to work with. So we have to do more of a "one size fits all" solution regarding tables.

Vinnie is not a fan of extra scrollbars within the document, so while adding scrolling is fine, try to hide or minimise the scrollbars themselves.

@cdw9 cdw9 moved this from Accepted to In Review in website-v2 Nov 18, 2024
@rbbeeston rbbeeston moved this from In Review to Done in website-v2 Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HTML/CSS The issue pertains to the html templates or stylesheets.
Projects
Status: Done
Development

No branches or pull requests

4 participants