firefox-ios icon indicating copy to clipboard operation
firefox-ios copied to clipboard

Remove FXIOS-14370 [Toolbar] Clean up TabToolbar

Open thatswinnie opened this issue 4 weeks ago • 4 comments

:scroll: Tickets

Jira ticket Github issue

:bulb: Description

Clean up TabToolbar from old toolbar.

:pencil: Checklist

  • [x] I filled in the ticket numbers and a description of my work
  • [x] 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 adding or modifying strings, I read the guidelines and will request a string review from l10n
  • [ ] If needed, I updated documentation and added comments to complex code

thatswinnie avatar Dec 08 '25 12:12 thatswinnie

Messages
:book: Project coverage: 38.03%

✍️ Strings Updated

Detected changes in Shared/Strings.swift. To keep strings up to standards, please add a member of the firefox-ios-l10n team as reviewer. 🌍

💪 Quality guardian

3 tests files modified. You're a champion of test coverage! 🚀

🏔️ Summit Climber

This PR is a big climb with 1365 lines changed! Thanks for taking on the heavy lifting 💪. Reviewers: a quick overview or walkthrough will make the ascent smoother.

🎉 BrowserViewController got smaller

Nice! BrowserViewController.swift got smaller by 41 lines.

❌ Per-file test coverage gate

The following changed file(s) are below 35.0% coverage:

File Coverage Required
firefox-ios/Client/Frontend/Browser/BrowserViewController/Extensions/BrowserViewController+ToolBarActionMenuDelegate.swift 6.0% 35.0%

Client.app: Coverage: 37.12

File Coverage
BrowserViewController.swift 29.22% ⚠️
TopTabsViewController.swift 21.52% ⚠️
PhotonActionSheetView.swift 0.0% ⚠️
BrowserViewController+ToolBarActionMenuDelegate.swift 5.97% ⚠️
OverlayModeManager.swift 88.64%
SingleActionViewModel.swift 48.0% ⚠️
TabTrayViewController.swift 49.51% ⚠️
PrivateModeButton.swift 81.82%

Generated by :no_entry_sign: Danger Swift against 3f497aa3147078ab99c0078cfc430ffbed0eed28

mobiletest-ci-bot avatar Dec 08 '25 12:12 mobiletest-ci-bot

I can reproduce the test failures reported on Bitrise.

On iPhone only: There's a split second that the toolbar isn't visible when the test attempts to navigate to private mode. Screenshot 2025-12-08 at 10 46 11

clarmso avatar Dec 08 '25 15:12 clarmso

This pull request has conflicts when rebasing. Could you fix it @thatswinnie? 🙏

mergify[bot] avatar Dec 08 '25 18:12 mergify[bot]

This pull request has conflicts when rebasing. Could you fix it @thatswinnie? 🙏

mergify[bot] avatar Dec 09 '25 13:12 mergify[bot]

🚀 PR merged to main, targeting version: 147.0

github-actions[bot] avatar Dec 11 '25 12:12 github-actions[bot]