Skip to content

Fix toolbar example - #344

Merged
deleonio merged 1 commit into
mainfrom
feature/toolbar-dokumentationsbeispiel-korrigieren
Jul 14, 2025
Merged

Fix toolbar example#344
deleonio merged 1 commit into
mainfrom
feature/toolbar-dokumentationsbeispiel-korrigieren

Conversation

@laske185

@laske185 laske185 commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Summary

  • fix toolbar example

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue and all changes relate to the issue
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

Copilot AI review requested due to automatic review settings July 14, 2025 12:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the toolbar example by correcting the JSON quoting of the _items prop in both the fenced code block and the inline usage snippet.

  • Updated _items attribute to use single quotes around the JSON string and double quotes inside.
  • Applied the same correction to the main example and the usage section.
Comments suppressed due to low confidence (2)

docs/30-components/toolbar.mdx:22

  • [nitpick] Since the toolbar has no child elements here, consider using a self-closing tag for brevity and to avoid unnecessary whitespace, e.g., <kol-toolbar ... />.
<kol-toolbar _label="Label" _items='[{"_label": "Button"}, {"_href": "#", "_label": "Link"}]'> </kol-toolbar>

docs/30-components/toolbar.mdx:29

  • [nitpick] You can simplify this snippet by converting it to a self-closing tag (e.g., />) since there are no nested children in the example.
        _items='[{"_label": "Button"}, {"_href": "#", "_label": "Link"}]'>

@github-actions

Copy link
Copy Markdown
Contributor

@laske185 laske185 linked an issue Jul 14, 2025 that may be closed by this pull request
@laske185
laske185 requested a review from deleonio July 14, 2025 12:37
@deleonio
deleonio merged commit 476cdef into main Jul 14, 2025
4 of 5 checks passed
@deleonio
deleonio deleted the feature/toolbar-dokumentationsbeispiel-korrigieren branch July 14, 2025 13:28
@publicuibot publicuibot Bot locked and limited conversation to collaborators Jul 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Toolbar - Doku überarbeiten

3 participants