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

[settings] Hide episode thumb if unseen (core wise) #14971

Merged
merged 1 commit into from
Dec 12, 2018

Conversation

DaVukovic
Copy link
Member

Description

This is a similar change to #14914, but done in core. I also followed a suggestion to change the wordings for that setting to make it less confusing. To be pretty clear, and for the reason we have another option for the "Show plot/information"-setting, I also decided to change the option labels to show exactly what is meant.

The only problem I can't solve myself (hence I added the "Help wanted" labe)l is, that the home screen TV show widgets don't change to the OverlaySpoiler image until the skin was reloaded (after playing a video, restarting Kodi, finish a database update, whatever). @Rechi helped me, but I don't get it ;). So I would love to see some help from someone who might be interested.

Motivation and Context

Same as for #14914

How Has This Been Tested?

Tested on Ubuntu 16.04

Screenshots (if appropriate):

See the PR mentioned above, The screenshots will be the same.

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • Clean up (non-breaking change which removes non-working, unmaintained functionality)
  • Improvement (non-breaking change which improves existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that will cause existing functionality to change)
  • Cosmetic change (non-breaking change that doesn't touch code)
  • None of the above (please explain below)

Checklist:

  • My code follows the Code Guidelines of this project
  • My change requires a change to the documentation, either Doxygen or wiki
  • I have updated the documentation accordingly
  • I have read the Contributing document
  • I have added tests to cover my change
  • All new and existing tests passed

@DaVukovic DaVukovic added Type: Improvement non-breaking change which improves existing functionality Component: Skin help wanted special GitHub label labels Nov 29, 2018
@DaVukovic
Copy link
Member Author

@notspiff I mentioned you in case you are interested. If not, then sorry for the noise.

@DaVukovic DaVukovic changed the title [Estuary] Hide episode thumb if unseen core wise [Estuary] Hide episode thumb if unseen (core wise) Nov 29, 2018
addons/resource.language.en_gb/resources/strings.po Outdated Show resolved Hide resolved
xbmc/video/VideoThumbLoader.cpp Outdated Show resolved Hide resolved
xbmc/video/VideoThumbLoader.cpp Outdated Show resolved Hide resolved
@DaVukovic DaVukovic force-pushed the Estuary-hidethumb-core branch 3 times, most recently from 775aabd to deee921 Compare November 29, 2018 20:04
@Hitcher
Copy link
Contributor

Hitcher commented Nov 30, 2018

@DaVukovic RE: The home screen widgets not updating.

This is actually a problem with home screen widgets in general (see below), not just specific to your PR, so this is good go in my opinion.

To reproduce for other widgets simply find a movie in your library that's displayed on the home screen in Estuary, change the poster and go back home. Old poster is still shown until you either refresh the skin or play a video.

@DaVukovic
Copy link
Member Author

DaVukovic commented Dec 1, 2018

@HitcherUK Yeah, I see what you mean. @Rechi PRd a hack to my branch, which I merged currently and I can confirm that announcing OnScanFinished after the change of that setting at least refreshes the widgets. Also a skin reload would do that. I know that this is a hack and probably not the method we will do that in the future, but I can confirm it working. The widgets are refreshed fine now and the setting has an immediate effect.

@Hitcher
Copy link
Contributor

Hitcher commented Dec 3, 2018

Thanks, this will make a lot of people happy.

@MartijnKaijser
Copy link
Member

Is this PR ok to be merged after its squashed?

@Rechi
Copy link
Member

Rechi commented Dec 7, 2018

No still needs a change, but @DaVukovic knows what to change.

@DaVukovic DaVukovic changed the title [Estuary] Hide episode thumb if unseen (core wise) [settings] Hide episode thumb if unseen (core wise) Dec 8, 2018
@DaVukovic
Copy link
Member Author

squashed the commits and also renamed the topic for a better match ;)

@DaVukovic DaVukovic force-pushed the Estuary-hidethumb-core branch 3 times, most recently from 26069fe to 53e26e5 Compare December 8, 2018 13:34
@Hitcher
Copy link
Contributor

Hitcher commented Dec 12, 2018

@MartijnKaijser I believe this is good to go now.

@MartijnKaijser MartijnKaijser merged commit 0665f06 into xbmc:master Dec 12, 2018
@DaVukovic DaVukovic deleted the Estuary-hidethumb-core branch December 12, 2018 17:51
@Rechi Rechi added this to the Leia 18.0-rc3 milestone Dec 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Settings Type: Improvement non-breaking change which improves existing functionality v18 Leia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants