Skip to content

Feat: Horizontal posters for search and recommendations. - #3144

Open
ByAyzen wants to merge 15 commits into
recloudstream:masterfrom
ByAyzen:ishorizontal-search-rec
Open

Feat: Horizontal posters for search and recommendations.#3144
ByAyzen wants to merge 15 commits into
recloudstream:masterfrom
ByAyzen:ishorizontal-search-rec

Conversation

@ByAyzen

@ByAyzen ByAyzen commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Since horizontal posters were only available on the main page, I wanted to bring this support to the search and recommendations sections as well. To not break multi search, I disabled it there and only applied it to extension search. I also used @prerelease tag to be sure it doesn't break the stable version.

Parameters for implementing to extensions:

@Prerelease
open val hasHorizontalSearch: Boolean = false
@Prerelease
open val hasHorizontalRecommendations: Boolean = false

I'm not completely sure about these two parameters because they can feel confusing; I can agree to change them. I could place them directly into search and new*****LoadResponse similar to how mainpage does it.

AI Usage

I used AI for implementing this idea. After that, I checked and cleaned up the issues.

Tests

I tested on both prerelease and debug versions. Tested for 2 days and didn't have any problems.

**Two comparison images below;

Search
Recommendation

@fire-light42 fire-light42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good feature, but needs more @Prerelease annotations and a small change in MainAPI 👍

* These horizontal options are only affecting extension search and load recommendations.
* */
@Prerelease
open val hasHorizontalSearch: Boolean = false

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These should rather be a property of SearchResponseList akin to HomePageList, to be consistent and allow for flexible results.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This can be made backwards compatible since it uses a builder function.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wasn't quite sure about these two properties either. Thanks for the feedback.

Comment thread library/src/commonMain/kotlin/com/lagradost/cloudstream3/MainAPI.kt
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.

3 participants