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

Reimplement rigboard page in a fully responsive manner #567

Merged
merged 3 commits into from
Dec 17, 2023

Conversation

FreneticScribbler
Copy link
Member

(cherry picked from commit b3939d8)

Requires refinement. Opening in order to get a review app going.

(cherry picked from commit b3939d8)

# Conflicts:
#	pipeline/source_assets/scss/dark_screen.scss
@FreneticScribbler FreneticScribbler changed the title Reimplement rigboard in a fully responsive manner Reimplement rigboard page in a fully responsive manner Oct 22, 2023
@FreneticScribbler FreneticScribbler marked this pull request as ready for review October 22, 2023 19:39
@FreneticScribbler FreneticScribbler temporarily deployed to pyrigs-pipel-subgrid-wzvslhqeo October 22, 2023 19:58 Inactive
@FreneticScribbler
Copy link
Member Author

FreneticScribbler commented Dec 17, 2023

I think this is now good to go if you're happy for me to merge it @nickw29 ? It's not perfect, and I bet someone's got a device that sits in a weird screen size spot, but it's better than the current sidescroll table by a mile!

Copy link

@nickw29 nickw29 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Send it

@FreneticScribbler FreneticScribbler merged commit 2a2ce74 into master Dec 17, 2023
3 checks passed
@FreneticScribbler
Copy link
Member Author

And so continues (barely) the long standing tradition of RIGS development during NAT panto 😆

@nickw29 nickw29 added the bug label Dec 17, 2023
@nickw29
Copy link

nickw29 commented Dec 17, 2023

Screenshot_20231217-184605
Small issue on Chrome on android

@FreneticScribbler
Copy link
Member Author

FreneticScribbler commented Dec 17, 2023 via email

@nickw29
Copy link

nickw29 commented Dec 17, 2023

I've gone back to a phone with a normal sized screen, 20:9 - 2400x1080 - 6.4"

Edit: Phone is a Pixel 6

@jerbzz
Copy link
Contributor

jerbzz commented Dec 17, 2023

FYI this is what it looks like on my iPhone 14 Pro Max (1284px × 2778px, 430px × 932px viewport, CSS Pixel Ratio of 3)

image

image

@FreneticScribbler
Copy link
Member Author

FreneticScribbler commented Dec 18, 2023

Looks good to me on your iPhone Jerbs, until you turn it landscape and then all manner of layout horrors fall out 😱 No idea why the header isn't full width. The rigboard should end where the new button is, as well, it's somehow leaking out of the page layout. Gak.

Nick, I was gonna guess that you were the one person in the data I pulled that had a weirdly ultra old version of Chrome, but I'm not sure that's allowed in Google Pixel land? If I set my viewport to the size it theoretically is on your phone, it doesn't break, so I'm a little stumped honestly!

@nickw29
Copy link

nickw29 commented Dec 18, 2023

Yeah, my phone and browser are up to date.
Mine also looks the same in landscape, maybe it's time for a forum poll

@nickw29
Copy link

nickw29 commented Jan 21, 2024

image
This is an interesting look

@FreneticScribbler
Copy link
Member Author

FreneticScribbler commented Jan 21, 2024 via email

@FreneticScribbler
Copy link
Member Author

Should be fixed momentarily. Still can't reproduce any of the other problems on either Firefox or Brave (chromium based) 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants