-
Notifications
You must be signed in to change notification settings - Fork 152
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
This line is misleading as a text field #158
Comments
@nb9960 What do you think? |
@vik4114 afaik that line is a mere partition between the two fields and is only for UI purposes. It has nothing to do with the textfields or symmetry. It's there to only get a general idea to users for where the divide lies between them. |
@ItsAdityaKSingh But without the line it looks good |
I guess the same follows for the other text fields too and because UI needs to be consistent, you'll have to make the changes throughout. |
@vik4114 Are you still working on this issue? |
@srockstech Yes, I was stuck with other work, I will raise pr soon. |
Hi vik can i work on this if possible ?? |
Hey, @vik4114, are you working on resolving this? |
I think no one solved this issue, so I created a PR(#170). So please check. @ItsAdityaKSingh |
is this issue resolved ?? |
Is your feature request related to a problem? Please describe.
This line is misleading as a text field and if it is acting as a partition still it is not symmetric
Screenshot
Describe the solution you'd like
I think we should remove it
Describe alternatives you've considered
-Else we can make it symmetric
-Or Bring it just below the text field
Additional context
Sol 1
Sol2
Are you working on this? (Yes/No)
Yes
The text was updated successfully, but these errors were encountered: