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

Show tables scrollable on mobile #2412

Closed
jakobroehrl opened this issue May 16, 2022 · 2 comments · Fixed by #3910
Closed

Show tables scrollable on mobile #2412

jakobroehrl opened this issue May 16, 2022 · 2 comments · Fixed by #3910

Comments

@jakobroehrl
Copy link

On small screens it's hard to see the table contents.
Maybe it's possible the make the table scrollable?

@teneighty-yvr
Copy link

First, thank you for adding table support! This is a fantastic feature for my organization!

I propose adding horizontal scrolling for all screen sizes. I was already using tables with the "plain text editor" application, but the first one I opened with this application after upgrading to NC 24 had a wide left margin, because the table is left aligned with the text area which is centered and 648 wide even though my display is 1920x1080, and the right edge was well past the right edge of the screen.

I also noticed that if I tab my way across the columns, it will show the last columns, but when I wrap around to the first column again, it does not return the table to it's original position. I actually can't see the first column at this point. Maybe that is a Firefox bug?

An important consideration for any fix is to utilize the entire available screen width effectively. I don't know how complicated that would be, given the document is contained in the narrower than my display div.ProseMirror. My HTML/CSS is pretty limited, but I think it's clear what I'm proposing?

Horizontal scrolling the whole document could also benefit plain text where, at the moment, lines are not wrapped so long lines have the same problem as tables.

my browser: Firefox 101.0

@mejo- mejo- added the high label Nov 11, 2022
@mejo- mejo- moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Nov 11, 2022
@jakobroehrl
Copy link
Author

Open this on mobile to see how github has done it:
nextcloud/collectives#482

mejo- added a commit that referenced this issue Mar 7, 2023
juliusknorr pushed a commit that referenced this issue Mar 10, 2023
nextcloud-command pushed a commit that referenced this issue Mar 10, 2023
Fixes: #2412

Signed-off-by: Jonas <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@github-project-automation github-project-automation bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Office team Mar 10, 2023
backportbot-nextcloud bot pushed a commit that referenced this issue Mar 10, 2023
Fixes: #2412

Signed-off-by: Jonas <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
nextcloud-command pushed a commit that referenced this issue Mar 10, 2023
Fixes: #2412

Signed-off-by: Jonas <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
mejo- added a commit that referenced this issue Mar 13, 2023
Fixes: #2412

Signed-off-by: Jonas <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants