broken on mobile #2
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
make me a PR that detects in JS if we're on a mobile device and doesn't start the checkers and doesn't render anything other than the header and description, and just renders a big box that says "Not yet available on mobile." (the box should render properly on mobile.). triple-check your work as i don't have an easy way of testing outside of production, so i'm just going to merge it and load the live site on my phone. do not break or change anything on non-mobile!
that PR doesn't close this issue, but i'll merge it to fix the site in prod right now, so people stop reporting that.
then, make me another PR that makes it actually work on mobile. you're going to have to totally redesign the per-host row layout. because we're going from a landscape to a portrait viewport, I think each host row needs to become quite tall - all of the left side data on top, then the sparkline full width below. you'll also need to fix the summary line at the top.
On it — will create two PRs:
Starting with PR 1 now.
On it. Two PRs coming:
Starting with PR 1 now.
Created two PRs for this:
Quick fix (merge to prod now): #3 — detects mobile and shows "Not yet available on mobile" message. No changes to desktop.
Full mobile layout: #5 — responsive redesign with stacked host rows for portrait viewports. Closes this issue on merge.