Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: regression: ion-textarea does not follow dir #29577

Open
3 tasks done
AmitMY opened this issue Jun 2, 2024 · 5 comments
Open
3 tasks done

bug: regression: ion-textarea does not follow dir #29577

AmitMY opened this issue Jun 2, 2024 · 5 comments
Assignees
Labels

Comments

@AmitMY
Copy link
Contributor

AmitMY commented Jun 2, 2024

Prerequisites

Ionic Framework Version

v8.x

Current Behavior

an ion-textarea with dir="auto" for Hebrew and Arabic, i son the left:
image

Expected Behavior

until a few weeks ago, it was on the right:
image

Steps to Reproduce

  <ion-textarea
    id="mobile"
    rows="1"
    dir="auto"></ion-textarea>

Code Reproduction URL

https://github.com/sign/translate

Ionic Info

Ionic:

Ionic CLI : 7.2.0

Utility:

cordova-res : 0.15.4
native-run : 2.0.1

System:

NodeJS : v22.0.0
npm : 10.5.1
OS : macOS Unknown

Additional Information

Current version 8.2.0

@brandyscarney
Copy link
Member

Thank you for the issue! Could you please provide a minimal reproduction of the issue? It's hard to see the problem when it is inside of an application. I have created a StackBlitz example here that you could fork and edit to show the problem: https://stackblitz.com/edit/mcqhu5?file=index.html

Alternatively, you can open a StackBlitz from any of our textarea documentation demos, fork them and modify them to show the problem. Thank you. 🙂

@brandyscarney brandyscarney added the ionitron: needs reproduction a code reproduction is needed from the issue author label Jun 14, 2024
Copy link

ionitron-bot bot commented Jun 14, 2024

Thanks for the issue! This issue has been labeled as needs reproduction. This label is added to issues that need a code reproduction.

Please reproduce this issue in an Ionic starter application and provide a way for us to access it (GitHub repo, StackBlitz, etc). Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.

If you have already provided a code snippet and are seeing this message, it is likely that the code snippet was not enough for our team to reproduce the issue.

For a guide on how to create a good reproduction, see our Contributing Guide.

@ionitron-bot ionitron-bot bot removed the triage label Jun 14, 2024
@AmitMY
Copy link
Contributor Author

AmitMY commented Jun 14, 2024

hi @brandyscarney ! long time

in this picture, it is auto, rtl, and auto.
the first line should behave like the second.
image

but dir=auto does not propagate to the child element
image

Please see https://stackblitz.com/edit/mcqhu5-2q1unz?file=index.html

@brandyscarney brandyscarney added triage and removed ionitron: needs reproduction a code reproduction is needed from the issue author labels Jun 24, 2024
@brandyscarney
Copy link
Member

Hey @AmitMY! It has been, good to see your name pop up! Sorry for taking so long to get back to you. I was on vacation last week. 🙂

Thanks for the updated StackBlitz. Do you remember when this regression started? Was it after a version update? I am seeing the same thing on v7: https://stackblitz.com/edit/mjxbz3?file=index.html

@brandyscarney brandyscarney added needs: reply the issue needs a response from the user and removed triage labels Jun 26, 2024
@AmitMY
Copy link
Contributor Author

AmitMY commented Jun 26, 2024

Not sure, possibly it was in ionic 6.
I have a screenshot service for my app (fastlane etc) and the current/expected behavior images are from it. that's where i noticed the bug.

Regardless of being a regression then, it is still a bug in v8 :(

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants