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

Rework section header layout for albums and episodes #968

Merged
merged 4 commits into from
Jan 21, 2024

Conversation

wutschel
Copy link
Contributor

@wutschel wutschel commented Jan 6, 2024

Description

This PR reworks the layout of the section header for albums and episodes. Now both use the same layout and method instead of separate implementations.

Overview of changes:

  • Move duplicate code into helper method
  • Unify layout for albums and seasons (adds fanart for season headers)
  • Correct and ease up the layout calculation
  • Simple runtime calculation and display
  • Replace magic numbers
  • Improve readability of bottom labels

Summary for release notes

Improvement: Rework section header layout for albums and episodes

@wutschel
Copy link
Contributor Author

wutschel commented Jan 7, 2024

Screenshots:
OjAwC.png

Copy link
Collaborator

@kambala-decapitator kambala-decapitator left a comment

Choose a reason for hiding this comment

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

Replace magic numbers

...and introduce new ones xD

XBMC Remote/DetailViewController.m Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
XBMC Remote/DetailViewController.m Outdated Show resolved Hide resolved
- Move duplicate code into helper method
- Unify layout for albums and seasons
- Correct and ease up the layout calculation
- Simple runtime calculation and display
- Replace magic numbers
Kodi uses 2:3 as aspect ratio for TV Show posters.
@wutschel
Copy link
Contributor Author

Squashed and rebased to master. Let me know if there's anything else to do.

@kambala-decapitator kambala-decapitator merged commit 1204115 into xbmc:master Jan 21, 2024
@wutschel wutschel deleted the rework_sectionheader branch January 21, 2024 08:44
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.

None yet

2 participants