Skip to content

Update README.md #2408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Update README.md #2408

wants to merge 1 commit into from

Conversation

do-me
Copy link

@do-me do-me commented Feb 9, 2025

Unfortunately it seems like fs doesn't work on iOS at the moment, see:

fs doesn't work on iOS at the moment, see tauri-apps/tauri#8843
@do-me do-me requested a review from a team as a code owner February 9, 2025 17:30
Copy link
Contributor

github-actions bot commented Feb 9, 2025

Package Changes Through b0f9ee6

There are 2 changes which include log with patch, log-js with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
api-example 2.0.17 2.0.18
api-example-js 2.0.13 2.0.14
log 2.2.1 2.2.2
log-js 2.2.1 2.2.2

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@FabianLars
Copy link
Member

This is not true. It's shit to work with, i give you that, but it still works at least a little bit. The apps i know of are all closed source with me not having access anymore so i can't give you an example right now, but this repo's example almost works (almost because the capabilities aren't correct).

If the file isn't in the application specific directories you'll have to combine it with the dialog plugin though. I'll see if i can push a minimal fix (no time for more) to the plugin.

@FabianLars FabianLars closed this Feb 9, 2025
@do-me
Copy link
Author

do-me commented Feb 9, 2025

Thanks a lot for taking the time, really appreciate it!
It's just that so far I didn't see a single working repo with fs on iOS yet, so it's kind of frustrating to poke around in the dark.

@FabianLars
Copy link
Member

yeah i totally get it. it's frustrating for me as well because i have no clue about iOS. I should have pushed the people who worked on mobile support more to work on docs/examples as well 🙃

@FabianLars
Copy link
Member

FabianLars commented Feb 9, 2025

@FabianLars
Copy link
Member

FabianLars commented Feb 9, 2025

oh damn that's a huge screenshot - obviously i don't have a clue about macOS as well 😂 - Edit: made it smaller...

@do-me
Copy link
Author

do-me commented Feb 9, 2025

Awesome, thanks a lot! Could you share the link to this example source code?

@FabianLars
Copy link
Member

https://github.com/tauri-apps/plugins-workspace/tree/v2/examples/api or more specifically, this is what i used in the screenshot: #2410

@do-me
Copy link
Author

do-me commented Feb 9, 2025

I reproduced it on my physical device, so fs indeed seems to work! Thank you very much for taking the time.
However, against this comment, out of all App* paths on iOS, AppCache is the only path that is not writable (forbidden path) when trying to mkdir (tauri-apps/tauri#8843 (comment)).
AppData seems to be the safer bet here.

By the way, the UX on mobile is a little inconvenient. A resize function for the console height would be great. Also, it'd be better if there was no automatic zoom on text input. I fixed this before in another app but don't remember what kind of setting I had to tweak.

@FabianLars
Copy link
Member

Also, it'd be better if there was no automatic zoom on text input.

100% - that drove me crazy. And as you can see in the screenshot, i couldn't fix it either 🙃

@do-me
Copy link
Author

do-me commented Feb 10, 2025

Also, it'd be better if there was no automatic zoom on text input.

100% - that drove me crazy. And as you can see in the screenshot, i couldn't fix it either 🙃

If I remember correctly, a simple event.preventDefault(); (see here https://github.com/do-me/tauri-basemap/blob/main/photon_geocode.js#L112) did the trick for me in this mini app: https://github.com/do-me/tauri-basemap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants