Skip to content

add multiple selection mode on browse source#1670

Closed
L-oung wants to merge 9 commits intomihonapp:mainfrom
L-oung:main
Closed

add multiple selection mode on browse source#1670
L-oung wants to merge 9 commits intomihonapp:mainfrom
L-oung:main

Conversation

@L-oung
Copy link

@L-oung L-oung commented Jan 27, 2025

No description provided.

@MajorTanya
Copy link
Member

What is this for?

@L-oung
Copy link
Author

L-oung commented Jan 27, 2025

What is this for?

Batch add manga from sources

@MajorTanya
Copy link
Member

MajorTanya commented Jan 27, 2025

What is this for?

Batch add manga from sources

But you can already add several of them individually? Just long press on an entry and it'll add it to your default category/show the category selection screen. This new feature doesn't seem worth it in that respect.

Edit: And even if the long press option didn't exist, you can still open the entry with a tap, then favourite it, then tap back and continue through the source's listings.

@L-oung
Copy link
Author

L-oung commented Jan 27, 2025

What is this for?

Batch add manga from sources

But you can already add several of them individually? Just long press on an entry and it'll add it to your default category/show the category selection screen. This new feature doesn't seem worth it in that respect.

Edit: And even if the long press option didn't exist, you can still open the entry with a tap, then favourite it, then tap back and continue through the source's listings.

Adding manga in large quantities is annoying:(

@L-oung
Copy link
Author

L-oung commented Jan 27, 2025

And the process of adding manga will become slower and slower. I dont know why

@cuong-tran
Copy link
Contributor

The last commit should go to a different PR, perhap?

@MajorTanya
Copy link
Member

It sure doesn't look related to this PR 🤔

L-oung and others added 2 commits February 3, 2025 17:11
  corner size
  book padding
  prefetch distance
  fetch items
Ui modified
@AntsyLich
Copy link
Member

Why is there entirely different feature changes in this PR?

@AntsyLich AntsyLich marked this pull request as draft February 24, 2025 18:45
L-oung and others added 4 commits April 17, 2025 15:13
# Conflicts:
#	app/src/main/java/eu/kanade/tachiyomi/ui/browse/source/browse/BrowseSourceScreen.kt
#	app/src/main/java/eu/kanade/tachiyomi/ui/browse/source/browse/BrowseSourceScreenModel.kt
#	data/src/main/java/tachiyomi/data/source/SourcePagingSource.kt
#	data/src/main/sqldelight/tachiyomi/migrations/4.sqm
#	i18n/src/commonMain/moko-resources/base/strings.xml
@MajorTanya
Copy link
Member

Are you just pushing your personal flavour changes onto the same branch you used to make the PR? At this point, the PR is basically unusable with all its different unrelated changes. Either clean up your branch to whittle it down to the actual feature you meant to implement or close the PR for good.

@L-oung L-oung closed this by deleting the head repository Apr 21, 2025
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.

4 participants