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

drop Downloads storyboard #2556

Merged
merged 7 commits into from
Apr 10, 2024
Merged

drop Downloads storyboard #2556

merged 7 commits into from
Apr 10, 2024

Conversation

mallexxx
Copy link
Collaborator

@mallexxx mallexxx commented Apr 5, 2024

Task/Issue URL: https://app.asana.com/0/1199230911884351/1206643583260394/f

Steps to test this PR:

  1. Validate downloads popover empty state, validate Open downloads folder link works
  2. Start a download instantly opening Downloads (Cmd+J), download should be added
  3. Validate progress, tooltips, button states, filename, icon match the downloaded file
  4. Validate each download action (button: cancel, restart, reveal); Context menu actions (open, reveal, cancel, open website, copy link, remove, cancel, restart, clear), clear all and open downloads button, open downloads folder link
  5. Validate when new downloads added the popover size is expanded, when removed - contracted

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@mallexxx mallexxx requested a review from SabrinaTardio April 5, 2024 12:59
Copy link
Contributor

github-actions bot commented Apr 5, 2024

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against 7b6b387

Copy link
Collaborator

@SabrinaTardio SabrinaTardio left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! Looks good!
Just two very minor comments

revealButton.setContentHuggingPriority(.defaultHigh, for: .vertical)
revealButton.alignment = .center
revealButton.bezelStyle = .shadowlessSquare
cancelButton.isBordered = false
Copy link
Collaborator

Choose a reason for hiding this comment

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

was it supposed to be reveal? although reveal is not showing a border anyway

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

copypasta

tableView.dataSource = self
tableView.menu = setUpContextMenu()

tableView.registerForDraggedTypes([.fileURL])
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe it’s nicer to keep as a separate func

@mallexxx mallexxx merged commit fd64be8 into main Apr 10, 2024
17 checks passed
@mallexxx mallexxx deleted the alex/dro-downloads-storyboard branch April 10, 2024 15:01
samsymons added a commit that referenced this pull request Apr 12, 2024
# By Dax the Duck (10) and others
# Via GitHub (6) and others
* main: (40 commits)
  Bump version to 1.83.0 (160)
  macOS VPN: Ask users to reboot if system extension was not uninstalled (#2603)
  macOS VPN: Ask users to reboot if system extension was not uninstalled (#2603)
  Fix popover not displayed reliably when VPN shortcut is unpinned (#2606)
  Automatically mark / close stale PRs (#2596)
  Update copy for DBP open button (#2586)
  Bump version to 1.83.0 (159)
  [Release PR] Fix lottie high Windowserver load (#2598)
  Bump version to 1.83.0 (158)
  BSK release 133.1.0 (#2597)
  Fix VPN bug: Nearest city breaks register requests (#2589)
  Fix lottie high Windowserver load (#2595)
  drop Downloads storyboard (#2556)
  Disable directory download (#2585)
  Add supported document types (#2581)
  Allow choosing downloads location in App Store builds (#2532)
  Fix Open Downloads not working (#2576)
  Update Privacy Dashboard URL on navigation commit (#2583)
  Percent-decode download filenames (#2584)
  Bump version to 1.83.0 (157)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
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