ionic-framework icon indicating copy to clipboard operation
ionic-framework copied to clipboard

bug: regression: `ion-textarea` does not follow `dir`

Open AmitMY opened this issue 1 year ago • 5 comments
trafficstars

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

AmitMY avatar Jun 02 '24 16:06 AmitMY

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 avatar Jun 14 '24 16:06 brandyscarney

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[bot] avatar Jun 14 '24 16:06 ionitron-bot[bot]

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

AmitMY avatar Jun 14 '24 17:06 AmitMY

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 avatar Jun 26 '24 19:06 brandyscarney

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 :(

AmitMY avatar Jun 26 '24 21:06 AmitMY

Thank you for submitting this issue! I was able to reproduce it. However, this is not a bug, as ion-textarea and ion-input have never supported this behavior. Instead, this falls under a feature request, similar to what ion-searchbar includes. Since an existing feature request already covers this, I’ll be closing this issue as a duplicate. I highly recommend following the feature request for updates.

thetaPC avatar Mar 11 '25 18:03 thetaPC

This has been resolved via PR https://github.com/ionic-team/ionic-framework/pull/30102 and will be available in an upcoming release of Ionic.

thetaPC avatar Mar 11 '25 23:03 thetaPC

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

ionitron-bot[bot] avatar Apr 10 '25 23:04 ionitron-bot[bot]