Skip to content

Mobile / Fix expanded column not being scrolled into view#2484

Merged
adjogima merged 2 commits intomainfrom
mobile/fix-board-scroll-position
Feb 5, 2026
Merged

Mobile / Fix expanded column not being scrolled into view#2484
adjogima merged 2 commits intomainfrom
mobile/fix-board-scroll-position

Conversation

@adjogima
Copy link
Contributor

@adjogima adjogima commented Feb 4, 2026

It's an iOS-only bug where navigating to a board would often fail to bring the expanded column centered into view.
Safari seems to have trouble honoring the scrollIntoView call, especially in the web views.

  • Start from the home screen.
  • Navigate to a board from the main menu
  • The expanded column is not properly scrolled and centered into view.
  • Pull to refresh fixes it.

I've played with disabling view transitions in the apps, adding a delay before triggering a scroll, implementing a custom scrollColumnIntoView function for this controller but nothing worked reliably.

What seems to work reliably is switching from smooth scrolling to instant in the apps.

@adjogima adjogima requested a review from jayohms February 4, 2026 17:45
@jayohms
Copy link
Contributor

jayohms commented Feb 5, 2026

@adjogima the change looks good to me. Does it feel ok when switching between columns now (after the board has loaded) that we don't have smooth scrolling?

@adjogima
Copy link
Contributor Author

adjogima commented Feb 5, 2026

@jayohms I went back and changed it to "instant" when restoring the previously expanded column but "smooth" when switching between columns.

Copy link
Contributor

@jayohms jayohms left a comment

Choose a reason for hiding this comment

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

Looks good, thanks.

@adjogima adjogima merged commit 9c7afcb into main Feb 5, 2026
12 checks passed
@adjogima adjogima deleted the mobile/fix-board-scroll-position branch February 5, 2026 16:47
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.

2 participants