• jadero@programming.dev
        link
        fedilink
        arrow-up
        20
        arrow-down
        1
        ·
        5 months ago

        There was a thread elsewhere asking whether a toggle should show current state or the state desired. There was enough disagreement that it quickly became apparent that, whatever else the toggle does, there should be something external to the toggle showing the possible states, indicating which way to move the toggle regardless of toggle appearance.

      • qaz@lemmy.world
        link
        fedilink
        arrow-up
        8
        ·
        5 months ago

        It did nothing when clicking it the first time and only changed the second time I clicked it

        • snowe@programming.devM
          link
          fedilink
          arrow-up
          6
          ·
          5 months ago

          I’ve wondered what this problem was for years but never cared to figure it out, because it always resolved after the first button press (just refresh the page and it all works properly). turns out it is something wrong with my use of local storage to save your theme state. if you don’t have the key in local storage then it does what you mentioned. I just need to switch this to prefers-color-scheme anyway.

      • fd93@programming.dev
        link
        fedilink
        arrow-up
        2
        ·
        4 months ago

        Hey - the poster isn’t actually the author. That would be me! Thanks for the feedback though. I normally just use Dark Reader for switching theme.

        • im sorry i broke the code@sh.itjust.works
          link
          fedilink
          arrow-up
          3
          ·
          edit-2
          5 months ago

          It works if you visited the website already but the first time it breaks: reloading fixes this while emptying the caches breaks it again. As another user pointed out the first click after emptying the cache doesn’t do anything at all even though the animation plays out just fine

          EDIT: I’m on safari ios, latest stable (17.3.1 if I’m not mistaken)