Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Footer #568

Closed
MeesD94 opened this issue Aug 21, 2024 · 3 comments · Fixed by #581 or #638
Closed

Footer #568

MeesD94 opened this issue Aug 21, 2024 · 3 comments · Fixed by #581 or #638
Assignees

Comments

@MeesD94
Copy link
Contributor

MeesD94 commented Aug 21, 2024

Kijk naar de column layout van Utrecht

@Rozerinay
Copy link
Contributor

Rozerinay commented Sep 9, 2024

Design review

  • Ik zie bij een container van ±860px dat de 1e kolom waar de Link list in zit, naast de pay-off 'De Rijksoverheid…' wordt geplaatst. Is het toevallig mogelijk om deze kolommen onder elkaar weer te geven bij deze containergrootte, net zoals op een small container?
  • Ik ben aan het zoeken naar waar de tokens voor gap en column vandaan komen. Deze zie ik niet terug in de Storybook van Utrecht. Waar zijn deze op gebaseerd?
  • color is nu rhc.color.wit in plaats van rhc.color.foreground.onEmphasis. Deze common token gebruiken we speciaal voor tekst op sterke achtergrondkleuren.

@AdhamAH AdhamAH assigned AdhamAH and unassigned Rozerinay Sep 12, 2024
@MeesD94 MeesD94 self-assigned this Sep 17, 2024
@Rozerinay
Copy link
Contributor

Rozerinay commented Sep 17, 2024

Design review 2

  • Zou je de breakpoint willen aanpassen naar 768px? Dan hebben we direct een logische breakpoint voor de tabletweergave.
  • Ik denk dat door het verwijderen van de spacing tokens er geen ruimte meer is tussen de columns. Blijkbaar waren dit tokens die jullie handmatig hadden toegevoegd, en dat was helemaal prima! Ik vroeg me alleen af waar het vandaan kwam. Het lijkt me verstandig om die tokens weer toe te voegen.
image

@MeesD94
Copy link
Contributor Author

MeesD94 commented Sep 17, 2024

@Rozerinay Ik had voordat ik vakantie ging dit component gemaakt. Die column en gap tokens had ik inderdaad toegevoegd voor de ruimte tussen de kolommen. Ik pak dit componenten weer verder op en zal ze terug zetten en de verdere feedback toepassen :)

@MeesD94 MeesD94 linked a pull request Sep 18, 2024 that will close this issue
@Rozerinay Rozerinay self-assigned this Sep 18, 2024
AdhamAH added a commit that referenced this issue Sep 26, 2024
#568 

- Breakpoint bij 860px 
- verwijder de `gap` en `column` tokens 
- De `color` is `rhc.color.wit`

---------

Co-authored-by: Ruben Smit <[email protected]>
Co-authored-by: Ruben Smit <[email protected]>
Co-authored-by: Rozerin <[email protected]>
Co-authored-by: Mees Work <[email protected]>
@Rozerinay Rozerinay moved this from Design review to Done in Community Sprint - Rijkshuisstijl componenten Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants