Summary: | Wrong font-sizes in the FireInfo module | ||
---|---|---|---|
Product: | Infrastructure | Reporter: | Maurice Gesswein <maurice.gesswein> |
Component: | Web Site | Assignee: | Sven Hoehn <shoehn> |
Status: | CLOSED NOTABUG | QA Contact: | Michael Tremer <michael.tremer> |
Severity: | - Unknown - | ||
Priority: | - Unknown - | CC: | shoehn |
Version: | unspecified | ||
Hardware: | unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | |||
Bug Blocks: | 11314 | ||
Attachments: | fireinfo_fonts |
Headlines like 'Latest Release' are still to small in M, S and XS. 10px instead of 14px and font-weight 300 instead of 400. Also the %-values are not vertically centered anymore. Looks perfect and L and XL. please check again font-size. Almost. At S and XS the headline like "Intel" should be 16px instead of 20px. Could you also replace "top: calc(50% - 20px)" with "top: calc(50% - 18px)", please? And one question about the font color of the percent values: I see "#546e7a" at the code. It should be "rgba:(38, 50, 56, 0.6);". Primary color with 60% opacity. We need transparency at the fonts when these are located on colored background. This will avoid to use additional colors for fonts. As in Zeplin sheet https://zpl.io/Z1LkANx these 6 colors for fonts will be all we need to cover almost all cases. Hint: You can click on the small gear next to "CSS" (hover of it) at Zeplin to change HEX to RGB and more options. please check again. On Zeplin I only see the rgb value but not the rgba value and therefor no transparency. |
Created attachment 515 [details] fireinfo_fonts Please recheck the font-sizes in the FireInfo module at viewport M, S & XS and compare with designs in Zeplin. See attached example of viewport M.