From 2fe6d7a10c8656de9a2e08e187486625d23c3535 Mon Sep 17 00:00:00 2001 From: William Carroll Date: Sun, 19 Apr 2020 13:05:55 +0100 Subject: Responsively size UI TL;DR: scale down UI for non-mobile devices. I pulled the screen resolution for my phone, the Google Pixel 4, off of the internet. I created a device profile in Chrome to develop this application specifically for my phone. To my surprise, when I opened the app on my phone, many of elements that looked good in Google Chrome, looked askew on my phone. I needed to troubleshoot. Here's how I did that: I used Tailwind to responsively color the bg for each breakpoint to see if my device was sm, md, lg, xl (according to Tailwind's breakpoint terminology). After reading Tailwind's documentation and comparing their breakpoints with my Pixel 4's width (i.e. 1080px), I figured that my device would be lg. It's not; it's md, which I confirmed by using ngrok to load localhost:8000 on my phone and see that the background-color was "md:bg-green-600". I'm still unsure why my device is not lg, but knowing that my device was md was enough to fix many of the styling issues. My current theory is that while my screen's resolution is 1080 wide, the pixel density affects the media query for the breakpoint. --- website/sandbox/learnpianochords/src/Tempo.elm | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'website/sandbox/learnpianochords/src/Tempo.elm') diff --git a/website/sandbox/learnpianochords/src/Tempo.elm b/website/sandbox/learnpianochords/src/Tempo.elm index 50485c4c0aba..a1afd776acb4 100644 --- a/website/sandbox/learnpianochords/src/Tempo.elm +++ b/website/sandbox/learnpianochords/src/Tempo.elm @@ -3,6 +3,7 @@ module Tempo exposing (render) import Html exposing (..) import Html.Attributes exposing (..) import Html.Events exposing (..) +import Responsive import UI @@ -15,7 +16,14 @@ type alias Props msg = render : Props msg -> Html msg render { tempo, handleInput } = div [ class "text-center" ] - [ p [ class "text-5xl py-10" ] [ text (String.fromInt tempo ++ " BPM") ] + [ p + [ [ "py-10" + , Responsive.h2 + ] + |> UI.tw + |> class + ] + [ text (String.fromInt tempo ++ " BPM") ] , UI.textField { placeholderText = "Set tempo..." , handleInput = handleInput -- cgit 1.4.1