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

[tvOS] SelectServerView Change to Menu #1363

Merged
merged 9 commits into from
Dec 19, 2024

Conversation

JPKribs
Copy link
Member

@JPKribs JPKribs commented Dec 16, 2024

Summary

Changes up some of the padding, coloring, and rounding of the FullScreenMenu. Enable FullScreenMenu as a ViewModifier to allow it to be used as View.fullScreenMenu. Create a modifier to the FullScreenMenu for orientation to allow the view to be either leading or trailing for more dynamic usage.

Clean up the SelectServerView to utilize the new FullScreenMenu and better mirror Apple's default Alert.

Videos

Old:

Old.mp4

New:

New.mp4

Screenshots

SelectServerView Buttons (Old vs New) Screenshot 2024-12-16 at 11 48 25
SelectServerView Server List (Old vs New) Screenshot 2024-12-16 at 11 49 59
FullScreenMenu Color Changes (Alert vs Old | Alert vs New) Screenshot 2024-12-16 at 11 57 51

@LePips
Copy link
Member

LePips commented Dec 16, 2024

Why not first investigating Menu? Moving to tvOS 17 as minimum to use Menu for this screen at the very least was a primary motivation.

@JPKribs
Copy link
Member Author

JPKribs commented Dec 16, 2024

I mostly went this direction since the Menu doesn't work from the Toolbar. I have a branch where I put this into a Menu which I think looks like this:

Simulator Screenshot - Apple TV 4K (3rd generation) - 2024-12-16 at 14 36 38

The only downside I see is the scroll scrolls with everything so we can't pin anything at the top like I did in my FullScreenMenu. Not the end of the world at all but just a note:

Simulator Screenshot - Apple TV 4K (3rd generation) - 2024-12-16 at 14 40 40

If you'd prefer, I can use this instead. This does mean there is no other usage of FullScreenMenu so would we want to delete that or do we think there is some potential usage elsewhere?

@JPKribs JPKribs changed the title [tvOS] Full Screen Menu Adjustments [tvOS] SelectServerView Cleanup Dec 16, 2024
@JPKribs JPKribs changed the title [tvOS] SelectServerView Cleanup [tvOS] SelectServerView Change to Menu Dec 16, 2024
@JPKribs
Copy link
Member Author

JPKribs commented Dec 17, 2024

I've updated this to be the Menu and I removed the 2 views that are no longer used. Actually a big fan of the Menu and it seems to be a lot sturdier on the emulator as well so I think that's the best way to go!

Let me know if you want me undo any of the deletions on this PR in case we need those views in the future.

@LePips LePips merged commit d580e8d into jellyfin:main Dec 19, 2024
4 checks passed
@JPKribs JPKribs deleted the tvOSFullScreenMenuCleanup branch December 19, 2024 22:45
ddrccw added a commit to ddrccw/Swiftfin that referenced this pull request Jan 26, 2025
* upstream/main: (392 commits)
  [tvOS] Add pin prompt to sign-in screen (jellyfin#1383)
  [iOS] Admin Dashboard - User Access Tags (jellyfin#1377)
  [Meta] 2025 Disclaimer (jellyfin#1381)
  [tvOS] Delete User from User Selection Screen (jellyfin#1359)
  [iOS] Media Item Menu - Identify Media Item (jellyfin#1369)
  [iOS] Admin Dashboard - User Profiles (jellyfin#1328)
  [iOS] Select all Users When Editing (jellyfin#1373)
  [Meta] Automatic String Organization (jellyfin#1372)
  [iOS & tvOS] Unused Localization Cleanup (jellyfin#1362)
  [tvOS] SelectServerView Change to Menu (jellyfin#1363)
  [tvOS] Update ConnectToServerView & UserSignInView (jellyfin#1365)
  Trim Fastlane Options (jellyfin#1367)
  Update Fastlane Runner (jellyfin#1366)
  [iOS & tvOS] Localize Existing Strings (jellyfin#1361)
  [iOS] Admin Dashboard - User Access Schedules (jellyfin#1358)
  [iOS] Admin Dashboard - Parental Ratings (jellyfin#1353)
  [iOS & tvOS] Error Cleanup (jellyfin#1357)
  update (jellyfin#1356)
  Fix possible duplicate ids (jellyfin#1354)
  [tvOS] Media Item Menu - Refresh / Delete Items (jellyfin#1348)
  ...

Signed-off-by: ddrccw <[email protected]>
@JPKribs JPKribs added the enhancement New feature or request label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants