Blockquote component (zie comment design review)

Robbert commented 3 months ago

Figma Blockquote in NLDS - Rijksoverheid - Bibliotheek

Rozerinay commented 3 months ago

PR: https://github.com/nl-design-system/rijkshuisstijl-community/pull/506

AlineNap commented 2 months ago

Nieuwe feature request voor de blockqoute:

We willen graag dat het mogelijk is om de nieuwe variant van de blockqoute te maken en de oude van de Fijkshuisstijl. Hiervoor moeten de volgende tokens worden toegevoegd aan het component.

utrecht/rhc/new.blockqoute.border-radius
blockquote.border.color
blockqoute.border-width

Daarnaast de mogelijkheid om wel of geen border aan bepaalde kanten te tonen, graag op basis van tokens zodat er van thema (oud/nieuw) geswitcht kan worden. Ik heb de verschillende varianten toegevoegd aan figma.

@Robbert heeft wellicht bovenstaande tokens al aan het utrecht component toegevoegd. Vraag of kijk of dat klopt, ze mogen sowieso op de utrecht code toegevoegd worden. De mogelijkheid om een border aan of uit te zetten en aan 1 kant een border-radius te hebben is wel Rijkshuisstijl specifiek.

En dan heb ik een vraag, hebben jullie de token utrecht.blockquote.row-gap toegevoegd aan het component? Of zit deze in het oorspronkelijke component van utrecht?

AlineNap commented 2 months ago

Een PR van mij met deze extra border tokens met een utrecht prefix is al gemerged. Gaat om deze: https://github.com/nl-design-system/rijkshuisstijl-community/pull/634

AlineNap commented 1 month ago

Feedback design review wordt verwerkt in:

AlineNap commented 1 week ago

Design review 15 November

1 appereance van de block-quote is genoeg, die kan de betreffende Rijksorganisatie dan vervolgens stijlen naar wens doormiddel van de tokens.
Tokens die wij zelf toevoegen graag de pre-fix rhc geven voor het onderscheid. Ik ga er vanuit dat we tokens hebben toegevoegd, als dat niet zo is let me know :).

Vragen

Ik zie dat er pakketjes zijn gemaakt om de verschillende varianten te creëren. Ik ben benieuwd hoe dit werkt, zijn dit overschrijvingen?
Zit het ticket van consistente oplossing keep hierin? (Ga zo dit ticket bekijken).

Opmerkingen

Als je tokens toevoegt is het fijn als je de type invult in de json. Dan komt het voor ons gelijk in de juiste categorie in token studio/ Figma. --> Al is het voor deze pakketjes wellicht juist wel goed dat ze apart staan bedenk ik me nu. Wacht even tot ik begrijp waarom deze pakketjes er zijn a la vraag 1 ;).
```
},
    "border-start-start-radius": {
      "value": "3rem",
      "type": "deze graag invullen, in dit geval: borderRadius"
    },
```

nl-design-system / rijkshuisstijl-community

Blockquote component (zie comment design review) #461

Design review 15 November