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.

Sign up for GitHub

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

Jump to bottom

[BUG] - Lack of RTL Support in Input #2315

Closed
mrbadri opened this issue Feb 5, 2024 · 1 comment · Fixed by #2316
Closed

[BUG] - Lack of RTL Support in Input #2315

mrbadri opened this issue Feb 5, 2024 · 1 comment · Fixed by #2316
Labels
🐛 Type: Bug Something isn't working

Comments

@mrbadri
Copy link
Contributor

mrbadri commented Feb 5, 2024

NextUI Version

2.0.10

Describe the bug

The repository does not work correctly when the direction is set to RTL (Right-to-Left). Elements are not displayed as expected, leading to usability issues.

The label is not in the right place

See Here

image

The Clear Button is not in the right place

See Here

image

Your Example Website or App

No response

Steps to Reproduce the Bug or Issue

  1. Go to [https://storybook.nextui.org/?path=/story/components-input--label-placement&globals=locale:ar-AE]
  2. Change the document direction to RTL (locale: Arabic).
  3. Observe the issues mentioned above.

Expected behavior

I expected the UI to properly support RTL direction, with elements aligning and displaying correctly from right to left.

Screenshots or Videos

No response

Operating System Version

All

Browser

Chrome
@mrbadri mrbadri added the 🐛 Type: Bug Something isn't working label Feb 5, 2024
mrbadri added a commit to mrbadri/nextui that referenced this issue Feb 5, 2024
@mrbadri
fix(input): resolve RTL support (nextui-org#2315)
8c0957f
@mrbadri mrbadri mentioned this issue Feb 5, 2024
Merged
@Aux9
Copy link

Aux9 commented Feb 15, 2024

Hi, I'm new on GitHub and this is my first comment and yes, I have this issue too!

mrbadri added a commit to mrbadri/nextui that referenced this issue Mar 9, 2024
@mrbadri
fix(input): add RTL support to the input component (nextui-org#2315)
687aeef
@mrbadri mrbadri mentioned this issue Mar 9, 2024
Closed
mrbadri added a commit to mrbadri/nextui that referenced this issue Mar 9, 2024
@mrbadri
fix(input): add RTL support to the kbd component (nextui-org#2315)
53a8d83
mrbadri added a commit to mrbadri/nextui that referenced this issue Mar 9, 2024
@mrbadri
fix(kbd): add RTL support to the kbd component (nextui-org#2315)
45da2e1
jrgarciadev pushed a commit that referenced this issue Mar 16, 2024
@mrbadri
fix(input): resolve RTL support (#2315) (#2316)
52dafd0 
* fix(input): resolve RTL support (#2315)

* fix: add changeset

* fix: add changeset for rtl input

* fix: convert changeset rtl input to ptach
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 Type: Bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@mrbadri @Aux9