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

Fix issue on rotating while text selected #3948

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Conversation

Bunn
Copy link
Contributor

@Bunn Bunn commented Feb 8, 2025

Task/Issue URL: https://app.asana.com/0/1204167627774280/1209357069596041

Description:
Fix a layout issue when rotating the device while the text was selected

Steps to test this PR:

  1. On the main branch, try to reproduce the issue with the steps here
  2. On this PR branch, do the same steps, check if it’s fixed.

Definition of Done (Internal Only):

@Bunn Bunn requested a review from brindy February 8, 2025 12:21
@@ -33,11 +33,11 @@ enum UniversalOmniBarState {
var clearTextOnStart: Bool { baseState.clearTextOnStart }
var allowsTrackersAnimation: Bool { baseState.allowsTrackersAnimation }
var showSearchLoupe: Bool { baseState.showSearchLoupe }
var showCancel: Bool { true }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The changes here helps with that flicking animation we saw on the simulator.

Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

Was unable to reproduce the bug. Tried rotating a bunch of times, backgrounding, foregrounding, rotating again, etc.

@Bunn Bunn merged commit 6450cdf into main Feb 10, 2025
16 checks passed
@Bunn Bunn deleted the bunn/aichat/ship-review-fixes branch February 10, 2025 12:06
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