Skip to content

Conversation

@JoaoFerreira-FrontEnd
Copy link
Contributor

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Sep 9, 2025

Issue number: resolves #30079


What is the current behavior?

Segments (IonSegment and IonSegmentView) do not work if placed on a dir="rtl" context. If click on button, it won't slide content of the next segment.

What is the new behavior?

  • calculate scroll value having into consideration the dir value;

Does this introduce a breaking change?

  • Yes
  • No

Other information

Preview

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added package: core @ionic/core package type: bug a confirmed bug report labels Sep 9, 2025
@vercel
Copy link

vercel bot commented Sep 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Sep 11, 2025 10:52am

- lint.fix;
- reconfig the basic-tests to look into rlt too;
Copy link
Member

@ShaneK ShaneK left a comment

Choose a reason for hiding this comment

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

This is a solid start, but I have some requested changes so that we can remain consistent in how we detect RTL and a bit of improved browser support.

Copy link
Member

@ShaneK ShaneK 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 to me, great work!

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added this pull request to the merge queue Sep 11, 2025
Merged via the queue into main with commit 66f517d Sep 11, 2025
51 checks passed
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd deleted the FW-6768 branch September 11, 2025 15: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 type: bug a confirmed bug report

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Swipeable Segments won't work with dir=RTL

3 participants