Skip to content

Conversation

@oscarcarlstrom
Copy link
Contributor

@oscarcarlstrom oscarcarlstrom commented Nov 18, 2025

Refactor the <LinkList> API to allow headings within link lists.

  • <LinkListItem> no longer supports link props, the component must now receive a <Link> as a child to which link props are passed. This makes for a more consistant and more granualar API.
  • Extracts all styling from the React implementation of <LinkList> to the tailwind package.
  • Removes the isExternal prop from the<LinkListItem>. External links are now identified byt the rel prop on the <Link> child (e.g <Link rel="external">). This is more consistent with how we control the DownloadIcon (download attribute), and it fits nicely with the <Link> API.
  • The <Link> component has also been extended with an internal _innerWrapper prop, similar to what we have in Heading component. This allows the <LinkList> to provide the correct icond for each link.
Screenshot 2025-11-18 at 13 11 33 Screenshot 2025-11-18 at 13 12 01

@oscarcarlstrom oscarcarlstrom requested a review from a team as a code owner November 18, 2025 12:11
@changeset-bot
Copy link

changeset-bot bot commented Nov 18, 2025

🦋 Changeset detected

Latest commit: 9976090

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@obosbbl/grunnmuren-react Patch
@obosbbl/grunnmuren-tailwind Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@oscarcarlstrom oscarcarlstrom merged commit 0f8cd6d into main Nov 20, 2025
1 check passed
@oscarcarlstrom oscarcarlstrom deleted the link-list-headings branch November 20, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants