[Discussion] Changing the way errors looks on editTexts

[cdhiraj40]

As the app already uses google's material TextInputLayout, it also should use its great-looking way to show errors.
there are places where showing error takes space in editText and it makes it look bad because of compression.
for example:

some screenshots of after using material TextInputLayout way to show error

CarSelectionAttributes Fragment	HSN/TSN Fragment

@SebaDro @asaikarthikeya what are your thoughts on this way to show an error?
would like to hear your thoughts
[using this issue as a discussion]

another thought: we can use discussions given by GitHub, so every time a person wants to request a new feature they can propose in discussions, and once approved they can create a new issue

I think it will help in decreasing the stale issues which are not required to work for now :)

[asaikarthikeya]

@cdhiraj40 hey, Yes. Your new proposed way of showing errors looks good to me. Showing error icon seem to be bit outdated and the compression is also observed in few devices. If you would like to work on this issue, please create a PR. Thanks.

[cdhiraj40]

@cdhiraj40 hey, Yes. Your new proposed way of showing errors looks good to me. Showing error icon seem to be bit outdated and the compression is also observed in few devices. If you would like to work on this issue, please create a PR. Thanks.

sure I would definitely like to work on this, you can assign me :)

[asaikarthikeya]

Generally for any issue, the PR which is raised first is considered. Other PR's which are created later will be not considered. As you have already mentioned your interest in this issue, any other developer should have to confirm with you before he works on this issue. So, No need to wait until it is assigned to you. You can work on any issue if it does not have a open PR ;) Thanks.

[pree-T]

May I work on it?

[cdhiraj40]

hey @pree-T I have already made PR for this #876
I would suggest you, find some issues which have no PRs linked to it :)