vf-core icon indicating copy to clipboard operation
vf-core copied to clipboard

Update design of the vf-breadcrumbs in figma and on the vf-site

Open Adedoyinebi opened this issue 1 year ago • 3 comments

From the review of the breadcrumbs design during the Intranet project. The Vf-breadcrumbs component's enhancement on desktop are as follows:

Improve the target area for the breadcrumb Increase the spacing after the “>” symbol from 4px to 8px to avoid overlap of touch target areas for small words in the breadcrumb

Task

Adedoyinebi avatar Apr 15 '24 14:04 Adedoyinebi

The desktop variants of the vf-breadcrumbs has been updated in figma

@stefangutnick kindly review this update

Adedoyinebi avatar May 10 '24 09:05 Adedoyinebi

@Adedoyinebi The padding now appears to be 4+4 px which is correct, looks good

stefangutnick avatar May 10 '24 12:05 stefangutnick

Based on the discussion with @bhushan-ebi yesterday, We agreed to use a padding of 8px on the left and right of the of the ">" Chevron for a uniform padding on both ends.

The padding on the top and bottom of each node has also been updated to 8px to achieve a target touch size of 44px for touch based interactions matching AAA accessibility standard on target touch area

Design of the Vf-breadcrumb has been updated in figma based on this.

@stefangutnick please review

Image below showing desired target touch area without overlap on other elements

Image

Adedoyinebi avatar May 16 '24 09:05 Adedoyinebi

Figma changes are done as per comment above https://github.com/visual-framework/vf-core/issues/2009#issuecomment-2104274387. Dev changes are done as part of parent ticket #1976

bhushan-ebi avatar May 22 '24 11:05 bhushan-ebi