Skip to content

Artist and author in track search info for anilist#1810

Merged
AntsyLich merged 10 commits intomihonapp:mainfrom
NarwhalHorns:artist-in-track-search-info
Mar 6, 2025
Merged

Artist and author in track search info for anilist#1810
AntsyLich merged 10 commits intomihonapp:mainfrom
NarwhalHorns:artist-in-track-search-info

Conversation

@NarwhalHorns
Copy link
Copy Markdown
Contributor

Include the artist and author information from anilist in the tracker search if available.

If all artists are the same as all authors it doesn't display artists.

Only thing i'm maybe concious of is that if ALL the info is there then it runs quite long and pushed the description quite low. Thought about putting the artist/authors on one row but don't think we have the space really.

But I think it's worth having this info on there.

I've put in a good amount of null protection cus I just don't know if those values are guaranteed from anilist query

image

@Animeboynz
Copy link
Copy Markdown
Contributor

Oh god that inconsistent gap is gonna annoy the living crap out of me
Screenshot_20250303_002826_Chrome

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

yeah... it's not great. Any ideas? Tbh we could probably discard score on this screen? It's not really relevant when searching for a track to bind

@MajorTanya
Copy link
Copy Markdown
Member

yeah... it's not great. Any ideas? Tbh we could probably discard score on this screen? It's not really relevant when searching for a track to bind

Score can be helpful to decide between different versions of the same manga, like when a tracker lists every volume separately as well as the main series, or when they list the pre-serialisation version separately from the serialised one.

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

don't really get how score helps with that but if you say so, shrink the text a little maybe?

@MajorTanya
Copy link
Copy Markdown
Member

Considering the space issues, maybe we can stick with just the author here? Also, please don't tell me AL returns full ISO 8601 timstamps for the dates.

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

Considering the space issues, maybe we can stick with just the author here?

I think it's very strange to keep score but remove artist but if that's what the people want lmao

Also, please don't tell me AL returns full ISO 8601 timstamps for the dates.

Nah that's just the preview values we have. the actual ones are like YYYY-MM-DD

@MajorTanya
Copy link
Copy Markdown
Member

Considering the space issues, maybe we can stick with just the author here?

I think it's very strange to keep score but remove artist but if that's what the people want lmao

Not wrong I guess. I'm not sure myself what the best option would be here.

Also, please don't tell me AL returns full ISO 8601 timstamps for the dates.

Nah that's just the preview values we have. the actual ones are like YYYY-MM-DD

😅

@Animeboynz
Copy link
Copy Markdown
Contributor

Animeboynz commented Mar 2, 2025

Not wrong I guess. I'm not sure myself what the best option would be here.

Redesign the composable, I believe in you two

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

NarwhalHorns commented Mar 3, 2025

huh, this is actually already a thing without the new items. I guess it's just rare to have all the data?
image

but now with everything lmaooo
image

@Animeboynz
Copy link
Copy Markdown
Contributor

huh, this is actually already a thing without the new items. I guess it's just rare to have all the data?

No yeah your right, I just checked in the app not every manga has all the details but the ones that do still push the description down (and its not even that uncommon with all these long as isekai titles making it so that the name stretched lines) but I guess with 2 more that's gonna be a lot more common sight

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

NarwhalHorns commented Mar 3, 2025

another option? list all creators just under the title?

image

tho we might wanna just think about redesigning this anyway cus that overflow, pushing down the desc is still possible regardless

@NarwhalHorns
Copy link
Copy Markdown
Contributor Author

If people are happy with that little add we can probably end this here and open someething else to think about redesigning it if that is desired.

@NarwhalHorns NarwhalHorns force-pushed the artist-in-track-search-info branch from 73a3ab0 to 7e9e94c Compare March 5, 2025 08:29
Comment thread app/src/main/java/eu/kanade/presentation/track/TrackerSearchPreviewProvider.kt Outdated
…eviewProvider.kt

Co-authored-by: AntsyLich <59261191+AntsyLich@users.noreply.github.com>
Comment thread app/src/main/java/eu/kanade/tachiyomi/data/track/anilist/dto/ALSearchItem.kt Outdated
Comment thread app/src/main/java/eu/kanade/tachiyomi/data/track/anilist/dto/ALManga.kt Outdated
Comment thread app/src/main/java/eu/kanade/presentation/track/TrackerSearch.kt Outdated
@NarwhalHorns NarwhalHorns requested a review from AntsyLich March 5, 2025 13:41
Copy link
Copy Markdown
Member

@AntsyLich AntsyLich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a changelog entry

Comment thread CHANGELOG.md Outdated
@NarwhalHorns NarwhalHorns requested a review from AntsyLich March 5, 2025 22:10
Comment thread CHANGELOG.md Outdated
Co-authored-by: AntsyLich <59261191+AntsyLich@users.noreply.github.com>
@NarwhalHorns NarwhalHorns requested a review from AntsyLich March 6, 2025 08:02
@AntsyLich AntsyLich enabled auto-merge (squash) March 6, 2025 08:02
@AntsyLich AntsyLich merged commit b702603 into mihonapp:main Mar 6, 2025
@AntsyLich AntsyLich mentioned this pull request Mar 6, 2025
4 tasks
cuong-tran added a commit to komikku-app/komikku that referenced this pull request Mar 7, 2025
…/mihon#1810)

Co-authored-by: Cuong-Tran <cuongtran.tm@gmail.com>
Co-authored-by: AntsyLich <59261191+AntsyLich@users.noreply.github.com>
(cherry picked from commit b702603)
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 9, 2025
@NarwhalHorns NarwhalHorns deleted the artist-in-track-search-info branch April 8, 2025 07:37
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.

4 participants