• limerod@reddthat.comM
    link
    fedilink
    English
    arrow-up
    2
    ·
    1 year ago

    You said the app is feature complete. But, if you find the time and energy to spend. You can probably try to add a dark theme that utilizes the "follow system" theme setting. And later if desired material you theming support. Otherwise, it does the job and is good as far as it can go.

    • russjr08@outpost.zeuslink.net
      link
      fedilink
      English
      arrow-up
      2
      ·
      1 year ago

      Dark theming is definitely a fair point, I'll definitely need to have a look into that.

      Material You support is probably going to be out of scope though, as I feel that would be making it complex for the sake of being complex.

      I do want to learn more about the Material You API that being said.

      • limerod@reddthat.comM
        link
        fedilink
        English
        arrow-up
        2
        ·
        1 year ago

        Hey, I read the top review of a samsung user. Even in my own testing, the app doesn't appear to extend the screen timeout on samsung. It locks out after the default timeout.

        • russjr08@outpost.zeuslink.net
          link
          fedilink
          English
          arrow-up
          2
          ·
          1 year ago

          Interesting, sounds like they're killing the wakelock that Caffeinate acquires then (which is what actually keeps the screen active), rather than killing the whole app itself.

          That's another one of those issues that I don't think there's too many workarounds for. Theoretically I might be able to have the app check to see if the wakelock is still active and if not, re-acquire it… but if there's no way for the app to "know" that the wakelock has been killed in the first place, the only way around it would be to constantly ask Android "Is the wakelock still active? Is the wakelock still active? Is the wakelock still active?" over and over again, which would definitely lead to battery issues.

          I do know it works on some Samsung devices, as I bought an old A2… something to test it on, and couldn't find any signs of a problem there.

          I mean hell, I'd love for there to be a way to not even require a wakelock for Caffeinate, but the only other way is a "soft" wakelock, in which you tell Android "Never turn the screen off while my app's window is open", but of course that would mean you'd need to keep the actual app window in the foreground and would defeat the whole purpose (such as my favorite usecase, keeping the screen on while I'm reading a recipe - or keeping the screen on while I'm tracking a delivery from a food delivery application).

          • limerod@reddthat.comM
            link
            fedilink
            English
            arrow-up
            2
            ·
            1 year ago

            I do know it works on some Samsung devices, as I bought an old A2… something to test it on, and couldn't find any signs of a problem there.

            I tested on A22 5g android 13.

            I mean hell, I'd love for there to be a way to not even require a wakelock for Caffeinate, but the only other way is a "soft" wakelock, in which you tell Android "Never turn the screen off while my app's window is open", but of course that would mean you'd need to keep the actual app window in the foreground and would defeat the whole purpose

            You could also do it so it requests the draw over other apps' permission and have a small UI element keeping the screen on(if it can work). But that could complicate things a bit further. The permission being misused(by malware), and it being restricted in secure places(banking apps, settings, etc). It would be an imperfect solution.