Skip to content

Conversation

@mapsandapps
Copy link
Contributor

@mapsandapps mapsandapps commented May 24, 2023

Issue number: #27422

⚠️ Give co-author credit to https://github.com/jon-pruessmeier on merge.


What is the current behavior?

Years in the date picker are displayed in descending order.

What is the new behavior?

  • Years in the date picker are displayed in ascending order
  • This matches the native behavior

Does this introduce a breaking change?

  • Yes
  • No

This is not a breaking change, but it may be unexpected and also may cause users' visual diff tests to fail. As such, we'll target this to 7.1

Other information

The updated snapshots show the change

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@github-actions github-actions bot added the package: core @ionic/core package label May 24, 2023
@mapsandapps mapsandapps changed the base branch from main to feature-7.1 May 25, 2023 15:20
@mapsandapps mapsandapps marked this pull request as ready for review May 25, 2023 17:56
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

LGTM

@mapsandapps mapsandapps merged commit 2098806 into feature-7.1 May 26, 2023
@mapsandapps mapsandapps deleted the FW-4305 branch May 26, 2023 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants