taskwarrior-webui icon indicating copy to clipboard operation
taskwarrior-webui copied to clipboard

Inoperable on mobile Safari and mobile Firefox on iOS

Open ethereal-engineer opened this issue 3 years ago • 4 comments

Here's what happens if you try to load this on your iPhone IMG_9603 It just spins. Nothing happens. Works great on Firefox on my linux machines and Mac laptop.

ethereal-engineer avatar Jan 27 '22 07:01 ethereal-engineer

I think this is the same bug mentioned in #20. It seems that some dependencies cause this problem but I'm not sure whether the recent commits fix them. You can try the latest version and see if it still occurs.

DCsunset avatar Feb 08 '22 03:02 DCsunset

Is there a docker image tag for the latest release? On 8 Feb 2022, 2:13 PM +1100, DCsunset @.***>, wrote:

I think this is the same bug mentioned in #20. It seems that some dependencies cause this problem but I'm not sure whether the recent commits fix them. You can try the latest version and see if it still occurs. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

ethereal-engineer avatar Feb 08 '22 05:02 ethereal-engineer

You can update image with the tag latest or use the new tag 20220208. They are the same images.

DCsunset avatar Feb 08 '22 13:02 DCsunset

Thank you! On 9 Feb 2022, 12:50 AM +1100, DCsunset @.***>, wrote:

You can update image with the tag latest or use the new tag 20220208. They are the same images. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

ethereal-engineer avatar Feb 08 '22 19:02 ethereal-engineer

This is still happening for me on iOS 16, both Safari and Firefox. I pulled down latest yesterday so it should have any of the dependency upgrades you mentioned in #20.

Fantastic tool otherwise. Works flawlessly with taskserver on desktop Firefox. Thanks for building it!

JoshMock avatar Sep 28 '22 02:09 JoshMock

Unfortunately, the problem still persists even in the latest version and the latest iOS. This is sad as my main use (90%+) would be iOS devices so at the moment I can’t actively deploy your otherwise fantastic solution.

super-ben avatar Oct 11 '22 10:10 super-ben

If it helps in any way, I found this when opening the site in Safari, MacOS:

ChunkLoadError: Loading chunk 2 failed. (missing: http://hostname:port/_nuxt/cf3ecde.js) promiseReactionJob

Also:

cf3ecde.js Line 2 invalid regular expression: invalid group specifier name

This tells me that the problem may be more easily solved by setting up that regular expression differently.

super-ben avatar Oct 13 '22 20:10 super-ben

It’d probably be easier to debug in an unminified JS bundle. I‘ll try to hop on a Mac soon and see if I can reproduce it in Safari. I tried setting up an iOS debugging session on my Linux desktop the other day using this but had trouble setting it up.

JoshMock avatar Oct 13 '22 20:10 JoshMock

Opened https://github.com/DCsunset/taskwarrior-webui/pull/48 to fix this cc @DCsunset.

JoshMock avatar Oct 15 '22 20:10 JoshMock

Hmm. Still happening after pulling the latest image and running from scratch. Can’t tell if my PR just didn’t fix it, if a service worker is caching old stuff, or if it’s an issue with the image building an outdated copy of the JS somehow.

I’ll keep debugging, but @DCsunset please reopen in the meantime.

JoshMock avatar Oct 16 '22 23:10 JoshMock

It appears to be a Docker image issue:

This works in Safari:

cd path/to/repo
docker build -t tw-webui .
docker run -p 50505:80 -v $HOME/.taskrc:/.taskrc -v $HOME/.task:/.task tw-webui:latest

This does not:

docker pull dcsunset/taskwarrior-webui:latest
docker run -p 50505:80 -v $HOME/.taskrc:/.taskrc -v $HOME/.task:/.task dcsunset/taskwarrior-webui:latest

The fact that the second one fails on a JS file cf3ecde.js is a strong clue: JS bundle file names change when their contents change, and latest still serves the same list of JS file names.

Not sure if you just need to re-run that docker push Github action or if something is wrong with the way it's configured that made it build and push old code. :thinking:

JoshMock avatar Oct 17 '22 00:10 JoshMock

It seems that two workflows were run at the same time yesterday. Maybe the earlier build overwrote the second one. I'll try rebuilding it again.

DCsunset avatar Oct 17 '22 15:10 DCsunset

In the latest build, the file cf3ecde.js doesn't exist anymore. Would you mind testing whether it works now @JoshMock ?

DCsunset avatar Oct 17 '22 17:10 DCsunset

Yep, it's working! Thanks for rebuilding @DCsunset. :black_heart:

I might open an issue for some responsive UI improvements now that I can see it on iOS. :laughing:

JoshMock avatar Oct 17 '22 19:10 JoshMock

That's great! Closing this issue now :D

DCsunset avatar Oct 17 '22 20:10 DCsunset