fix(datetime): time picker now scrolls to correct value

[liamdebeasi]

Pull request checklist

Please check if your PR fulfills the following requirements:

• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)
• Build (npm run build) was run locally and any changes were pushed
• Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

Issue URL: resolves #24878

The problem is that scrollActiveItemIntoView is called before isColumnVisible is set to true: https://github.com/ionic-team/ionic-framework/blob/main/core/src/components/picker-column-internal/picker-column-internal.tsx#L100

This did not used to be a problem, but with the introduction of the isColumnVisible check in https://github.com/ionic-team/ionic-framework/blob/main/core/src/components/picker-column-internal/picker-column-internal.tsx#L152, this caused the initial scroll to no longer happen.

What is the new behavior?

• Moved isColumnVisible to be set to true before calling the scroll operation.

Does this introduce a breaking change?

• Yes
• No

Other information

I opted not to add a new test as we do already have a test for this, but the test suite that checks is not working. I am working on fixing the test suite with my Playwright migration.

[sean-perkins]

Awesome, thanks for fixing this 👍