Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@mallexxx
Copy link
Collaborator

@mallexxx mallexxx commented Feb 8, 2024

Task/Issue URL: https://app.asana.com/0/1201037661562251/1206542455948422/f

Description:

  • String-initialized Colors and Images replaced with compiler-generated asset names
  • Moved local packages assets into the package resources

Steps to test this PR:

  1. Validate common sanity running the regular smoke test

Internal references:

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

@github-actions github-actions bot added the bot: not in app board Added by automation for pull requests with tasks not added to macOS App Board Asana project label Feb 8, 2024
<customView translatesAutoresizingMaskIntoConstraints="NO" id="efe-Pc-ueP" userLabel="Window Dragging View" customClass="WindowDraggingView" customModule="DuckDuckGo_Privacy_Browser" customModuleProvider="target">
<rect key="frame" x="0.0" y="0.0" width="845" height="38"/>
</customView>
<customView hidden="YES" translatesAutoresizingMaskIntoConstraints="NO" id="yvS-k9-opP" customClass="GradientView" customModule="DuckDuckGo_Privacy_Browser" customModuleProvider="target">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tab Bar gradient dropped in #1357

@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2024

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

Generated by 🚫 dangerJS against 87ed665

@ayoy ayoy removed the bot: not in app board Added by automation for pull requests with tasks not added to macOS App Board Asana project label Mar 4, 2024
@ayoy
Copy link
Collaborator

ayoy commented Mar 4, 2024

@mallexxx could you please resolve conflicts? 😬 Seeing that I feel sorry it took me so long to get to this one.

@ayoy ayoy self-requested a review March 4, 2024 13:10
@ayoy ayoy self-assigned this Mar 4, 2024
Copy link
Collaborator

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM, this is a wonderful improvement! 👏 Smoke testing went well, nothing outstanding caught my eye. I've played a bit with adding new assets and renaming and it works great. Thanks a lot @mallexxx!

@ayoy ayoy assigned mallexxx and unassigned ayoy Mar 5, 2024
@mallexxx mallexxx merged commit 478ac06 into main Mar 5, 2024
@mallexxx mallexxx deleted the alex/generated-asset-names branch March 5, 2024 15:00
samsymons added a commit that referenced this pull request Mar 6, 2024
# By Alexey Martemyanov (10) and others
# Via GitHub
* main: (39 commits)
  Add Peoplewhiz data broker (#2310)
  Integrate confirm entitlements endpoint for macOS App Store purchase (#2325)
  Make dbSaveBloomFilterError pixel daily and continuous (#2299)
  Subscription UI updates (#2303)
  Add email pattern on opt-out pixel (#2307)
  Use compile-time checked generated asset names (#2177)
  fix bundle name in breakByRaisingSigInt (#2319)
  BSK update for iOS autofill support to delete all passwords (#2254)
  Bump BrowserServicesKit (#2309)
  Bump version to 1.78.0 (132)
  Fix crash on popup window display; adjust naming (#2315)
  Bump version to 1.78.0 (131)
  Set marketing version to 1.78.0
  Update embedded files
  Fix parsing commit log (#2311)
  DBP: Set correct duration of submit-success pixel (#2308)
  Publish subsequent internal releases right after building (#2306)
  DBP disable pixel changes (#2293)
  Remove duplicated broker items (#2294)
  fix tab title not updated (#2302)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants