Skip to content
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

Many problems at first side loading of a ZiM file #4067

Closed
kelson42 opened this issue Nov 8, 2024 · 5 comments · Fixed by #4075
Closed

Many problems at first side loading of a ZiM file #4067

kelson42 opened this issue Nov 8, 2024 · 5 comments · Fixed by #4075
Assignees
Labels
Milestone

Comments

@kelson42
Copy link
Collaborator

kelson42 commented Nov 8, 2024

Image

Here the points:

  • It seems the message in the screenshot is displayed only once at first. I wonder if it would not be best to just show the usual one "do you want to move or copy..." and write an explanation in the help.
  • Ater closing copy, it goes straight to copy... but actually it should propose the chooser of storage (I have a SD card)
@kelson42 kelson42 added the bug label Nov 8, 2024
@kelson42 kelson42 added this to the 3.12.0 milestone Nov 8, 2024
@MohitMaliFtechiz
Copy link
Collaborator

MohitMaliFtechiz commented Nov 13, 2024

It seems the message in the screenshot is displayed only once at first. I wonder if it would not be best to just show the usual one "do you want to move or copy..." and write an explanation in the help.

@kelson42 good idea, because now, we have improved the copy/move dialog message(#4069) so users can better understand this. Also, we will add a cancel button so that users have more control over this functionality.

Ater closing copy, it goes straight to copy... but actually it should propose the chooser of storage (I have a SD card)

The copied ZIM file will go into the current selected storage #3941 (comment) (I remember I had a conversation about this with you). For example:- if the user has selected "internal storage" in the settings screen the file will go into the "internal storage", the same for the SD card. If there is not enough storage available it will ask to configure the storage(for example if there is not enough storage in internal storage it asks to configure the storage to an SD card).

But thinking about your opinion, it is better to provide the option for selecting the storage before copy/move since it will let users directly copy/move the files in their wanted storage without configuring the storage from settings.

Edited: @kelson42 Do you want to show the chooser of storage once at the initial copy/move operation or do you want to show it on every copy/move operation? To me you want only to show the chooser of storage at the initial copy/move operation, please correct me if I am wrong.

@kelson42
Copy link
Collaborator Author

@MohitMaliFtechiz Thx, you get things right. The storage chooser should only be displayed once at start (and only if there is a chiice to make).

@kelson42
Copy link
Collaborator Author

@MohitMaliFtechiz Once CI green, please rebase and merge and release for testing in PS

@MohitMaliFtechiz
Copy link
Collaborator

@MohitMaliFtechiz Once CI green, please rebase and merge and release for testing in PS

@kelson42 I can not directly merge the PR, it needs your approval. The PR is now ready for review with all the videos and screenshots.

@MohitMaliFtechiz
Copy link
Collaborator

@kelson42 The application has been released in testing on PS. https://github.com/kiwix/kiwix-android/actions/runs/11837938150/job/32985980177.

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

Successfully merging a pull request may close this issue.

2 participants