Skip to content

remove downloads storyboard and other minor clean up#4458

Open
brindy wants to merge 1 commit intomainfrom
brindy/remove-downloads-storyboard
Open

remove downloads storyboard and other minor clean up#4458
brindy wants to merge 1 commit intomainfrom
brindy/remove-downloads-storyboard

Conversation

@brindy
Copy link
Copy Markdown
Contributor

@brindy brindy commented Apr 15, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1200194497630846/task/1213916260872783?focus=true
Tech Design URL:
CC:

Description

Remove downloads storyboard.

Testing Steps

  1. Run the app and check downloads screen opens.
  2. Download a file make sure it appears in downloads
  3. Delete the file, make sure downloads updates

Impact and Risks

Low: Minor visual changes, small bug fixes, improvement to existing features

What could go wrong?

Quality Considerations

Notes to Reviewer


Internal references:

Definition of Done | Engineering Expectations | Tech Design Template


Note

Medium Risk
Mostly a UI navigation refactor, but it also upgrades content-scope-scripts to a new major version which could affect content blocking behavior at runtime.

Overview
Removes Downloads.storyboard from the project and updates segueToDownloads() to present DownloadsListHostingController() directly instead of instantiating a storyboard.

Updates DownloadsListHostingController to be initializer-driven (no init(coder:) support) and performs minor appearance setup cleanup. Separately bumps the SwiftPM content-scope-scripts dependency from 13.41.0 to 14.0.0 in Package.resolved.

Reviewed by Cursor Bugbot for commit b0ade0a. Bugbot is set up for automated code reviews on this repo. Configure here.

@brindy brindy requested a review from dus7 April 15, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant