Skip to content

PCIOS-649: Smart Folders Suggestion: Navigation bar visible#4388

Draft
pocketcasts wants to merge 1 commit into
trunkfrom
pcios-649/hide-nav-bar-in-suggested-folders-view
Draft

PCIOS-649: Smart Folders Suggestion: Navigation bar visible#4388
pocketcasts wants to merge 1 commit into
trunkfrom
pcios-649/hide-nav-bar-in-suggested-folders-view

Conversation

@pocketcasts

Copy link
Copy Markdown
Contributor

Resolves https://linear.app/a8c/issue/PCIOS-649/smart-folders-suggestion-navigation-bar-visible

Summary

Hide the navigation bar in SuggestedFoldersView and move the close button from the navigation toolbar into the view body.

Changes

  • Added .navigationBarHidden(true) to hide the empty navigation bar on the suggested folders screen
  • Moved the close button from a ToolbarItem into an HStack at the top of the view body
  • Removed the now-unnecessary .navigationBarTitleDisplayMode(.inline) modifier

Verification

  • Lint: PASS
  • Tests: N/A — pre-existing CarPlay build errors unrelated to this change
  • Diff review: PASS — confirmed changes are scoped to the single file

Confidence

HIGH — Straightforward UI modifier change. The navigation bar was the only source of the empty bar, and the close button has been relocated to maintain functionality.


This PR was created autonomously by linear-solver.
Triage complexity: simple | Linear issue

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dangermattic

Copy link
Copy Markdown
Collaborator
1 Warning
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
1 Message
📖 This PR is still a Draft: some checks will be skipped.

Generated by 🚫 Danger

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.

2 participants