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

Make pod-tables scroll horizontally on smaller views #2260

Merged
merged 1 commit into from Aug 13, 2018

Conversation

MorayJ
Copy link
Contributor

@MorayJ MorayJ commented Aug 13, 2018

If a table is too wide for a page a horizontal scroll bar will be added to the bottom.

If the page is focused, the right key will scroll right and left to come back.

I'm presuming this will work on mobile with finger dragging, but no testing set up for that at the moment.

Reverted a previous commit doing this, and this should now work. 'nowrap' was causing the problem which is not in this commit.

The Problem

Squished table cells on mobile #650

Table columns that are outside the viewport are not viewable. Problem on mobile. This occurs in the bug report and is also in the 'Routines' page, directly linkable from the front page of the site making the bug quite prominent.

The Solution

Add css via sass generator following advice from someone's question on stackoverflow (https://stackoverflow.com/questions/5533636/add-horizontal-scrollbar-to-html-table) on best way for creating scrollable table.

If a table is too wide for a page a horizontal scroll bar will be added to the bottom.

If the page is focused, the right key will scroll right and left to come back.

I'm presuming this will work on mobile with finger dragging, but no testing set up for that at the moment.

Reverted a previous commit doing this, and this should now work.
@MorayJ MorayJ merged commit 5f1eb21 into Raku:master Aug 13, 2018
@MorayJ MorayJ deleted the pod-table-scroll branch August 20, 2018 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant