Bugfix FXIOS-11307 [Homepage] [TopSites] top site layout on rotation

[cyndichin]

📜 Tickets

Jira ticket
Github issue

💡 Description

Fix how we determine the number of tiles for top sites. This issue only exists with the Toolbar Redesign disabled. This is because we are dispatching more actions with the Toolbar Redesign so it causes the numberOfTilesPerRow to re-calculate more often. The issue without the toolbar redesign is that we call the calculation less often and the last call was calculated with the wrong size.

This change guarantees that we are calculating the proper number of tiles before the view changes and adding another dispatch causes the datasource to be updated.

📝 Checklist

You have to check all boxes before merging

• Filled in the above information (tickets numbers and description of your work)
• Updated the PR name to follow our PR naming guidelines
• Wrote unit tests and/or ensured the tests suite is passing
• When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
• If needed, I updated documentation / comments for complex code and public methods
• If needed, added a backport comment (example @Mergifyio backport release/v120)

Screenshots

Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-02-07.at.10.25.30.mp4

[cyndichin]

It feels odd to me to not have the number of tiles as payload associated with the dispatch action. But I kept the changes simple instead of passing the int around. Let me know what you think!