grommet / hpe-design-system

HPE Design System
45 stars 23 forks source link

Fix figma Dateinput components #3736

Closed SeamusLeonardHPE closed 4 months ago

SeamusLeonardHPE commented 4 months ago

Multiple issues have been uncovered with the existing date input components.

Sizing & icon alignment across multiple variants:

Image

KennyAtHPE commented 4 months ago

Figma branch

Requesting review @SeamusLeonardHPE @ashifalinadaf

I have gone ahead and updated the dateinput component based on user feedback and findings. THE FOLLOWING CHANGES ARE NOT BREAKING CHANGES UNLESS SPECIFIED

DateInput changes: Input field


TO AVOID BREAKING CHANGES, I HAVE NOT UPDATED:

ashifalinadaf commented 4 months ago

Reviewed and left some comments on the figma reviewing. @KennyAtHPE

KennyAtHPE commented 4 months ago

@ashifalinadaf @SeamusLeonardHPE made all the changes you both recommended. Requesting just ONE reviewer from you two before merging.

SeamusLeonardHPE commented 4 months ago

Hey Kenny, 2 issues I see:

1- The "icon right" visibility boolean is applied to all variants expect the default. (could is be removed completely? surely the button is required to launch the calendar overlay?) 2- The "range" visibility prop is applied to some Date:Single text inputs

Dropped comments on figma also.

image image

KennyAtHPE commented 4 months ago

Changes made and merged. Changes listed in #3767. Closing ticket