project-lux / lux-frontend

Web front end of LUX
Apache License 2.0
3 stars 0 forks source link

Issue with font in Chrome only (from 893) #13

Closed gigamorph closed 4 months ago

gigamorph commented 8 months ago

Problem Description:

Expected Behavior/Solution: Shouldn't do this. Looks like font doesn't support these diacritics in Chrome. What are the options? First priority if fixing the error/image. Deprioritizing the bold glitch.

Requirements: List of details required for the completion of the issue or requirements for the feature/bug. This can also include requirements that lie outside of the teams such as new design docs or clarification from an outside source.

Needed for promotion: If an item on the list is not needed, it should be crossed off but not removed.

UAT/LUX Examples: image image

Dependencies/Blocks:

Related Github Issues:

Related links: Bugherds:

Wireframe/Mockup: Place wireframe/mockup for the proposed solution at end of ticket.

roamye commented 6 months ago

From Kam in old tix: "Update 11/29/23: It appears the font LUX is using does not support diacritics in Google Chrome but other browsers are supported. Adding a fallback font would mean the current default font would be replaced in favor of a diacritic supported font in cases where that default font does not support the diacritics. However, this could impact styling. Below is an example of the diacritic supported font vs the font that does not support diacritics. The supported font is smaller compared to the unsupported font. This would have to be a universal change meaning the font size impact would be throughout the app and would also have to be considered.

NOTE: In the two screenshots, the computed font size for each header text is 49.6px or 3.1em but appears smaller.

Supported: Image

Unsupported: Image "

roamye commented 6 months ago

Per comment above should this issue be proposed-closed?

I can bring to ticket forming for discussion but if it is better suited for IT meeting let me know.

@kamerynB @prowns

kamerynB commented 6 months ago

@roamye I think I need to configure backup fonts in the frontend. I have to discuss this with @gigamorph again. He had a better understanding of this than I did. This should remain open until I confirm that.

kamerynB commented 5 months ago

I've moved this to Prioritization because it just requires added a backup font.

@prowns @roamye @jffcamp

roamye commented 4 months ago

Approved by UAT. Will need to be tested by someone with Chrome as the ones in tst/prod have the updated change for Windows.

kamerynB commented 4 months ago

@roamye This appears to be fine in Chrome on Mac. Refer to the screenshots attached.

Screenshot 2024-06-14 at 9 02 57 AM Screenshot 2024-06-14 at 9 03 09 AM
roamye commented 4 months ago

Both test and prod functioned correctly for windows so a mac user was needed to test the UAT. Seong and Xinjian have macs and were able to confirm this issue functions as it should in PROD with chrome. Closing.