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

feat(backdrop): add backdrop support #105

Merged
merged 2 commits into from
Sep 26, 2020
Merged

feat(backdrop): add backdrop support #105

merged 2 commits into from
Sep 26, 2020

Conversation

heyhippari
Copy link
Contributor

A component has been added to the default layout, which displays an image from a Vuex store.

The Vuex store can either set or clear backdrops.

To set a backdrop, you must pass a BaseItemDto[] in order to retrieve the relevant images. It is designed to allow a rotation of the array, in order to change which backdrop is displayed.

Note: It's currently only displayed on the item details page. It looks kind of bad, so we might want to redesign the page a bit to make use of the backdrops.

@heyhippari heyhippari force-pushed the feat/backdrops branch 2 times, most recently from 960a340 to 1dbb2f2 Compare September 20, 2020 05:31
layouts/default.vue Outdated Show resolved Hide resolved
components/Backdrop.vue Outdated Show resolved Hide resolved
@heyhippari
Copy link
Contributor Author

I'm actually going to rework this a bit to use Blurhash as backdrops instead, so putting it back into draft.

@heyhippari heyhippari marked this pull request as draft September 24, 2020 07:41
@camc314
Copy link
Contributor

camc314 commented Sep 24, 2020

Another thing you could do is still use the picture. But have a customizable blur amount in the settings

@heyhippari
Copy link
Contributor Author

The issue is that blurs are really expensive, especially on images that big.

Not all devices we'll run on are super fast (Smart TVs are one we have already confirmed can't handle the blur filter, over on jf-web, but cheap 100$ Android phones and cheap laptops come to mind as well).

It's sort of nice to have for customization, but imo the performance hit isn't really worth it and a blurred image provides much better visual separation, making it easier to read text and such.

@heyhippari heyhippari mentioned this pull request Sep 25, 2020
90 tasks
@heyhippari heyhippari force-pushed the feat/backdrops branch 3 times, most recently from 612f9c2 to 3803e9c Compare September 25, 2020 12:09
A component has been added to the default layout, which displays a blurhash from a Vuex store. The
Vuex store can either set or clear backdrops. To set a backdrop, you must pass a BaseItemDto[] in
order to retrieve the relevant images.
@heyhippari heyhippari marked this pull request as ready for review September 25, 2020 12:14
@heyhippari
Copy link
Contributor Author

This is ready for reviews again @camc314.

I reworked the store to use actions instead of a plugin (going to make the same changes to the other stores in an upcoming PR), since that's how it's supposed to be done.

It also uses blurhashes now, since that looks much cleaner.

Copy link
Contributor

@camc314 camc314 left a comment

Choose a reason for hiding this comment

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

Cool, this looks good.

Do you think it'd be a good idea to apply a filter (lighten/darken) depending on the theme?

@sonarcloud
Copy link

sonarcloud bot commented Sep 25, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@heyhippari
Copy link
Contributor Author

Cool, this looks good.

Do you think it'd be a good idea to apply a filter (lighten/darken) depending on the theme?

I changed a bit how the overlay color works (It didn't react to the light theme before).

There is now a "background" property on the Vuetify themes. Vuetify will generate a bunch of CSS classes for it automatically.

It's now used as the background color for the app and for the overlay. I took the background colors of the current Jellyfin Light and Dark themes for now.

@camc314 camc314 merged commit 61966da into master Sep 26, 2020
@camc314 camc314 deleted the feat/backdrops branch September 26, 2020 16:51
@heyhippari heyhippari added this to the Preview Release 1 milestone Dec 6, 2020
Feature parity with Jellyfin Web automation moved this from Done to Reviewer approved Dec 7, 2020
Feature parity with Jellyfin Web automation moved this from Reviewer approved to Done Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants