Skip to content

Bugfix FXIOS-12489 [Tab tray UI experiment] Disable swipe when drag and drop happens #27225

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

Merged
merged 2 commits into from
Jun 11, 2025

Conversation

lmarceau
Copy link
Contributor

📜 Tickets

Jira ticket
Github issue

💡 Description

Disabling the swipe when we are drag and dropping a tab cell. It breaks the tab tray otherwise.

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If needed, I updated documentation and added comments to complex code
  • If needed, I added a backport comment (example @Mergifyio backport release/v120)

@lmarceau lmarceau requested a review from a team as a code owner June 10, 2025 17:58
@lmarceau lmarceau requested a review from Foxbolts June 10, 2025 17:58
@lmarceau
Copy link
Contributor Author

@Mergifyio backport release/v140

Copy link
Contributor

mergify bot commented Jun 10, 2025

backport release/v140

✅ Backports have been created

@mobiletest-ci-bot
Copy link

Messages
📖 Project coverage: 37.18%
📖 Edited 7 files
📖 Created 1 files

Client.app: Coverage: 35.42

File Coverage
TabDisplayPanelViewController.swift 26.07% ⚠️
TabDisplayView.swift 33.88% ⚠️
TabTrayViewController.swift 50.71%
TabTrayCoordinator.swift 92.86%

Generated by 🚫 Danger Swift against 9a91db2

@lmarceau lmarceau merged commit fe1e53e into main Jun 11, 2025
8 checks passed
@lmarceau lmarceau deleted the lm/FXIOS-12489-#27224-disable-swipe-drag-and-drop branch June 11, 2025 17:57
mergify bot pushed a commit that referenced this pull request Jun 11, 2025
…nd drop happens (#27225)

* Disable swipe when drag and drop happens

* Adjust unit tests

(cherry picked from commit fe1e53e)

# Conflicts:
#	firefox-ios/Client.xcodeproj/project.pbxproj
lmarceau added a commit that referenced this pull request Jun 11, 2025
…nd drop happens (backport #27225) (#27260)

* Bugfix FXIOS-12489 [Tab tray UI experiment] Disable swipe when drag and drop happens (#27225)

* Disable swipe when drag and drop happens

* Adjust unit tests

(cherry picked from commit fe1e53e)

# Conflicts:
#	firefox-ios/Client.xcodeproj/project.pbxproj

* Fix conflicts

---------

Co-authored-by: lmarceau <[email protected]>
lmarceau added a commit that referenced this pull request Jun 12, 2025
…n drag and drop happens (#27225)"

This reverts commit fe1e53e.
lmarceau added a commit that referenced this pull request Jun 12, 2025
…27292)

* Revert "Bugfix FXIOS-12489 [Tab tray UI experiment] Disable swipe when drag and drop happens (#27225)"

This reverts commit fe1e53e.

* Disable drag and drop on tab tray with experiment
mergify bot pushed a commit that referenced this pull request Jun 12, 2025
…27292)

* Revert "Bugfix FXIOS-12489 [Tab tray UI experiment] Disable swipe when drag and drop happens (#27225)"

This reverts commit fe1e53e.

* Disable drag and drop on tab tray with experiment

(cherry picked from commit 9563b5c)

# Conflicts:
#	firefox-ios/Client.xcodeproj/project.pbxproj
lmarceau added a commit that referenced this pull request Jun 12, 2025
…backport #27292) (#27294)

* Refactor FXIOS-12520 [Tab tray UI experiment] Disable drag and drop (#27292)

* Revert "Bugfix FXIOS-12489 [Tab tray UI experiment] Disable swipe when drag and drop happens (#27225)"

This reverts commit fe1e53e.

* Disable drag and drop on tab tray with experiment

(cherry picked from commit 9563b5c)

# Conflicts:
#	firefox-ios/Client.xcodeproj/project.pbxproj

* Fix conflict

---------

Co-authored-by: lmarceau <[email protected]>
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.

3 participants