ui-components icon indicating copy to clipboard operation
ui-components copied to clipboard

fix(#1438): update app header and microsite header padding

Open vanessatran-ddi opened this issue 1 year ago • 2 comments

  • Update padding for microsite header
  • Make sure alignment between the microsite header and app header exists.

Demo link: https://vanessatran-ddi.github.io/ui-components/#/

Video:

https://github.com/GovAlta/ui-components/assets/120135417/11361ab7-6af9-4df6-b1ef-ea2136632044

vanessatran-ddi avatar May 03 '24 19:05 vanessatran-ddi

@vanessatran-ddi A few issues have been found:

  • [x] Small Screen size -- Padding right side AppHeader -- should be using the variable --goa-spacing-m, it's using 1rem right now
  • [x] Medium Screen size -- Padding right side AppHeader -- should be using the variable --goa-spacing-xl, it's using the same size as Small
  • [x] Large Screen size -- Padding right side AppHeader -- completely wrong, there's no padding at all

All of the above problems I noticed are because, unlike the Microsite Header which applies padding to the whole element, for the AppHeader, we're applying padding to elements inside the AppHeader, which feels strange and wrong.

ArakTaiRoth avatar May 03 '24 22:05 ArakTaiRoth

@vanessatran-ddi A few issues have been found:

  • [x] Small Screen size -- Padding right side AppHeader -- should be using the variable --goa-spacing-m, it's using 1rem right now
  • [x] Medium Screen size -- Padding right side AppHeader -- should be using the variable --goa-spacing-xl, it's using the same size as Small
  • [x] Large Screen size -- Padding right side AppHeader -- completely wrong, there's no padding at all

All of the above problems I noticed are because, unlike the Microsite Header which applies padding to the whole element, for the AppHeader, we're applying padding to elements inside the AppHeader, which feels strange and wrong.

Small screen: image Medium screen (Ipad Air) image Desktop image

vanessatran-ddi avatar May 06 '24 15:05 vanessatran-ddi

:tada: This PR is included in version 1.17.0-alpha.49 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar May 15 '24 23:05 tzuge

:tada: This PR is included in version 4.17.0-alpha.25 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar May 21 '24 16:05 tzuge

:tada: This PR is included in version 1.21.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar Jun 11 '24 21:06 tzuge

:tada: This PR is included in version 4.21.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar Jun 11 '24 21:06 tzuge

:tada: This PR is included in version 3.0.0-alpha.4 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar Jun 27 '24 16:06 tzuge

:tada: This PR is included in version 3.0.3 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

tzuge avatar Jun 27 '24 17:06 tzuge