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

Locale: Exclamation mark should be displayed on the right-hand side in all languages #5119

Closed
cpinatel7 opened this issue Apr 22, 2021 · 13 comments · Fixed by #8001
Closed
Assignees
Labels
focus: translation For translation issues type: bug 🐛 [2] Velocity rating (Fibonacci)

Comments

@cpinatel7
Copy link

Exclamation mark should be displayed on the right-hand side in all languages

  • English: Exclamation mark is displayed on the right-hand side
  • Languages: Exclamation mark is displayed on the left-hand side

Example: English vs Russian

ExclamationMark

From GRC dev:

We have reproduced the issue in below fork code:

https://github.com/JangleShubham/enterprise-ng.git

Please go to http://localhost:4200/ids-enterprise-ng-demo/tabs-vertical after running this project and go into Contract tab, create validation error, it will show exclamation error icon at left side.

Please see attachment for more detail.

image-2021-04-22-14-24-01-867

Please ask enterprise team to follow above steps to reproduce the issue.

@tmcconechy tmcconechy added [2] Velocity rating (Fibonacci) type: bug 🐛 labels Apr 22, 2021
@tmcconechy tmcconechy changed the title Exclamation mark should be displayed on the right-hand side in all languages Locale: Exclamation mark should be displayed on the right-hand side in all languages Apr 22, 2021
@tmcconechy
Copy link
Member

As a note i can't reproduce this on the example we have:

https://main-enterprise.demo.design.infor.com/components/validation/test-error-on-tab-vertical.html?locale=ru-RU
and
https://main-enterprise.demo.design.infor.com/components/validation/test-error-on-tab-vertical.html?locale=en-US

I cant seem to run your project its giving a ton of errors.... Perhaps you can put it in https://stackblitz.com/edit/ids-quick-start-940 (make a fork and add the code). Or perhaps you can try a newer version?

@cpinatel7
Copy link
Author

From: Shubham Jangle shubham.jangle@infor.com
Sent: vendredi 23 avril 2021 05:49
To: Christophe Louis Pinatel Christophe.Pinatel@infor.com
Subject: RE: [infor-design/enterprise] Locale: Exclamation mark should be displayed on the right-hand side in all languages (#5119)

Hello Christophe,

I guess issue is there in provided links by Tim
https://main-enterprise.demo.design.infor.com/components/validation/test-error-on-tab-vertical.html?locale=ru-RU
and
https://main-enterprise.demo.design.infor.com/components/validation/test-error-on-tab-vertical.html?locale=en-US
For all languages including en-US exclamation error icon is appearing at left side in above examples and also in GRC as well.

Can you please check? Or the issue is something different(I might be thinking of something else)

Thanks,
Shubham Jangle

@tmcconechy
Copy link
Member

Yes the icon seems to appear on the left side? Is there anything wrong with that? For example anything from a translation point of view?

@cpinatel7
Copy link
Author

It's a minor issue but there is no justification / requirement to display ! on the right in English and on the left in languages. It should be on the right-hand side. Thanks

@tmcconechy tmcconechy added priority: trival Least important issues and removed status: cant reproduce labels Apr 23, 2021
@tmcconechy
Copy link
Member

Ok agreed it probably makes sense on the right

@ghost ghost added the focus: translation For translation issues label Apr 28, 2021
@tmcconechy tmcconechy removed the priority: trival Least important issues label May 24, 2022
@stale
Copy link

stale bot commented May 18, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Marked as 360 days old. Will close if not needed. label May 18, 2023
@cpinatel7
Copy link
Author

Still valid. Thanks

@stale stale bot removed the stale Marked as 360 days old. Will close if not needed. label May 18, 2023
@tmcconechy
Copy link
Member

@cpinatel7 i think we need to know what languages in our set should be on the right and which should be on the left?
Is this a true statement: Its on the left in English and EVERY other language its on the right?

@cpinatel7
Copy link
Author

I think we need to understand the logic first. Example: In English? Why one ! on the left and one ! on the right? What's the logic? Thanks

image

@tmcconechy
Copy link
Member

@cpinatel7 there really isnt any logic to these. But for the input field i think its common to put the error at the front of the text unless its Right To Left. But for the tab error we could put it on either side (design desicion).

So i think for both cases we should just have the error on the left unless its a RTL language? Unless there are language concerns with either?

One other factor is there needs to be room for it on the left so thats why in some cases we stick it on the right.

From your side can you see any translation recommendations on this as im not really sure we have logic and its well thought out so we can do it how makes sense

@cpinatel7
Copy link
Author

cpinatel7 commented Oct 2, 2023

My recommendation is to be consistent. Same approach in all languages including English. In short, I agree with your statement "i think for both cases we should just have the error on the left unless its a RTL language".

I think an exclamation point sign in red triangle would be better than the current icon (the current icon is quite similar to a standard exclamation mark so positioning it on the left is a bit confusing)

image

@ericangeles
Copy link
Contributor

@tmcconechy, do you think we should ask for design feedback here?

@tmcconechy
Copy link
Member

For the icon change i dont think we want to change the icon. I think its as expected. But we can check with @talkaboutdesign what do you think about making the tab errors on the left?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: translation For translation issues type: bug 🐛 [2] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants