google / site-kit-wp

Site Kit is a one-stop solution for WordPress users to use everything Google has to offer to make them successful on the web.
https://sitekit.withgoogle.com
Apache License 2.0
1.25k stars 291 forks source link

Difference in Font size for Enhance toggle description #8820

Closed wpdarren closed 4 months ago

wpdarren commented 5 months ago

Bug Description

As reported by @kelvinballoo on the Asana bug bash board

The font size for the Enhance Toggle description is 12px, it should be 14px, similar to all the other text in that expanded module.

image

Figma says it's 14px anyway:

image


Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation Brief

Test Coverage

QA Brief

Changelog entry

eugene-manuilov commented 4 months ago

IB ✔️

mohitwp commented 4 months ago

QA Update ⚠️

@benbowler As per QAB `font size and font color for the Enhanced conversion tracking Toggle description should match the figma design . But, In the Analytics edit settings view, the 'Enable Enhanced Conversion Tracking' toggle description has a font size of 14px, which matches the Figma design. However, the font color differs from the Figma file, where the color is #6C726E. I believe the current implementation aligns with the existing styling. Can you please confirm if the font color should also be changed to #6C726E?

Figma Analytics- ![image](https://github.com/google/site-kit-wp/assets/94359491/8717d7fb-be7c-4f9d-afad-cfc8a7c70d2e) Analytics module - ![image](https://github.com/google/site-kit-wp/assets/94359491/6fac92d9-7b18-4444-ba3d-6fe8e8ccccf8)

Pass cases

Ads module- ![image](https://github.com/google/site-kit-wp/assets/94359491/dfa836fc-94dc-4503-982b-ac2501575d53) Analytics module - ![image](https://github.com/google/site-kit-wp/assets/94359491/6fac92d9-7b18-4444-ba3d-6fe8e8ccccf8)
benbowler commented 4 months ago

Hey @mohitwp, good point, this ticket was originally about the font size. I suggest with create a follow up ticket, something like "Unify settings helper text color", were we address this for all settings screens, otherwise we could get into a situation where there are differences between different toggles/different settings screens, what do you think?

mohitwp commented 4 months ago

QA Update ✅

Thanks @benbowler for the confirmation ! Yes, I have same thought that for consistency current implementation is correct. I will create separate ticket to implement consistency under settings page.

Ads module- ![image](https://github.com/google/site-kit-wp/assets/94359491/dfa836fc-94dc-4503-982b-ac2501575d53) Analytics module - ![image](https://github.com/google/site-kit-wp/assets/94359491/6fac92d9-7b18-4444-ba3d-6fe8e8ccccf8)