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

default language for browse source. #554

Merged
merged 16 commits into from
Feb 26, 2023
Merged

default language for browse source. #554

merged 16 commits into from
Feb 26, 2023

Conversation

nyagami
Copy link
Member

@nyagami nyagami commented Feb 18, 2023

To test this. please clear all all app data

@nyagami
Copy link
Member Author

nyagami commented Feb 19, 2023

of course for images fetching

nyagami and others added 13 commits February 20, 2023 21:26
some sources should have headers when fetch image data
- Saving images in binary form, not utf-8 base64 so that gallery etc can see them
- Downloads work with new WebViewReader's image loading features
- Added loading icons when images are loading asynchronusely
- Fix promise rejection when trying to overwrite image file (shouldn't happen in production, but it miraculously did in dev)
- Fix CSS class 'loadIcon' not being applied to downladed chapter image loader icons and loader spinners not spinning
- Made loader spinners smaller. Now they look "a bit better" (according to my friend whom I showed the spinners)
fix(download): Download chapter image fixes
@nyagami
Copy link
Member Author

nyagami commented Feb 23, 2023

Done job.
From now, if there's any source have trouble with resources (mostly images). It should have headers (option) defined to bypass cloudflare (example: Hako). Otherwise, dont use headers

@rajarsheechatterjee rajarsheechatterjee merged commit 3289f35 into LNReader:main Feb 26, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants