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

Conversation

@mallexxx
Copy link
Collaborator

Task/Issue URL: https://app.asana.com/0/1177771139624306/1204335652578292/f

Description:

  • Fixes saved Popup window frame affecting New WIndow size
  • Fixed conflicting Address Bar constraints leading to purple window frame

Steps to test this PR:

  1. open popup window using console->window.open("duckduckgo.com")->open blocked popup
  2. Resize&close the popup
  3. Open new window, validate the new window does have a size of its parent window, not the recently closed popup size

Internal references:

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

@mallexxx mallexxx requested a review from SabrinaTardio August 23, 2023 10:29
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.

Nice one! 🥇 LGTM

@mallexxx mallexxx merged commit 3391ce8 into develop Aug 24, 2023
@mallexxx mallexxx deleted the alex/fix-window-frame-saving branch August 24, 2023 09:08
samsymons added a commit that referenced this pull request Aug 25, 2023
# By Alexey Martemyanov (2) and others
# Via Diego Rey Mendez (1) and GitHub (1)
* develop:
  Update DBP BSK version (#1548)
  Add DBP package  (#1482)
  disable 2 flaky tests (#1543)
  Update BSK with autofill 8.2.0 (#1536)
  iOS InContext Email Protection Signup support (#1419)
  fix windows repositioned on sleep/display on-off (#1533)
  Move pixel sender logic into the main view controller (#1528)
  Updated metadata and rules for 1.53.2
  fix saved popup frame applied to new window (#1530)

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