-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
TextField issue with diacritics not fully visible #3983
Comments
Looks like this might be font related. I tried a TextField inside a Provider with the arabic locale, it's closer but the third line under that letter still seems cut off. https://codesandbox.io/s/broken-dust-8fscze?file=/src/App.js It looks like this happens in native as well https://jsfiddle.net/pbw3L0kv/2/ Will bring up with our design, I tried changing the line-height and padding with partial luck. Changing the line-height rendered all the diacritic in the vertical space, but was still cut off in the horizontal. |
After a discussion with the team. We've realized that codesandbox is not part of our typekit provisioning, nor possibly is chromatic. As a result, the images we've been dealing with and example apps are not actually correct. We need to use an environment where we actually resolve to the correct font and see if this is still a problem there. We have another issue, neither our docs site nor localhost appears to be provisioned for the Arabic Adobe font face, so we'll need to add that, and any others, such as Hebrew We also need to verify this against other accent marks that hang below the text baseline, such as the French |
Adobe Arabic is sans-serif and we want a serif font for consistency, so we're using Myriad Arabic. The same for Hebrew. Spectrum CSS has made this change as well. |
Here is a textfield in an Arabic locale provider with Arabic text, using Myriad Arabic. These screen shots were all generated from https://reactspectrum.blob.core.windows.net/reactspectrum/4a41f3b71c62796056043e36972fa7118f2a0e3e/storybook/index.html?path=/story/textfield--default&providerSwitcher-express=false
FireFox - everything handled correctly Safari - field looks fine, but label arabic diacritic is cut off |
🐛 Bug Report
full diacritics not visible in TextField for some characters depending on the browser
🤔 Expected Behavior
full diacritics should be visible
😯 Current Behavior
💁 Possible Solution
🔦 Context
💻 Code Sample
https://32w4px.csb.app/
🌍 Your Environment
this happens in some browsers but not others - mainly Firefox seems to be the only one to handle correctly
🧢 Your Company/Team
AEP
🕷 Tracking Issue (optional)
The text was updated successfully, but these errors were encountered: