Skip to content

Add Splitter component documentation link#2191

Merged
rkaraivanov merged 4 commits intomasterfrom
radomirchev-add-splitter
Apr 20, 2026
Merged

Add Splitter component documentation link#2191
rkaraivanov merged 4 commits intomasterfrom
radomirchev-add-splitter

Conversation

@radomirchev
Copy link
Copy Markdown
Contributor

Description

Provide a clear and concise summary of the changes in this PR and the motivation behind them

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Refactoring (code improvements without functional changes)

Related Issues

Closes #

Testing

Checklist

  • My code follows the project's coding standards
  • I have tested my changes locally
  • I have updated documentation if needed
  • Breaking changes are documented in the description

@radomirchev radomirchev requested a review from kdinev April 17, 2026 12:24
@radomirchev radomirchev self-assigned this Apr 17, 2026
@radomirchev radomirchev added the documentation Improvements or additions to documentation label Apr 17, 2026
Copilot AI review requested due to automatic review settings April 17, 2026 12:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates the project’s root README component matrix to include a documentation link entry for the Splitter component.

Changes:

  • Added a Splitter row to the Components table with a Docs reference and released version.
  • Added a [Splitter Docs] reference link target in the README link definitions.

Comment thread README.md Outdated
| Toast | ✅ | [Docs][Toast Docs] | [2.1.0] | [MIT](LICENSE) |
| Rating | ✅ | [Docs][Rating Docs] | [2.1.0] | [MIT](LICENSE) |
| Slider | ✅ | [Docs][Slider Docs] | [2.0.0] | [MIT](LICENSE) |
| Splitter | ✅ | [Docs][Splitter Docs] | [7.1.0] | [MIT](LICENSE) |
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

The table references [7.1.0] for Splitter, but there is no corresponding link definition ([7.1.0]: ...) in the reference section below, so the released-version link will render as a broken reference. Add a [7.1.0] release link definition consistent with the other versions.

Copilot uses AI. Check for mistakes.
Comment thread README.md Outdated
| Toast | ✅ | [Docs][Toast Docs] | [2.1.0] | [MIT](LICENSE) |
| Rating | ✅ | [Docs][Rating Docs] | [2.1.0] | [MIT](LICENSE) |
| Slider | ✅ | [Docs][Slider Docs] | [2.0.0] | [MIT](LICENSE) |
| Splitter | ✅ | [Docs][Splitter Docs] | [7.1.0] | [MIT](LICENSE) |
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

The components table appears to be ordered by released version (descending). Adding Splitter (7.1.0) down in the 2.x section breaks that ordering and makes the list harder to scan. Consider moving this row near the top with the other 6.x/5.x entries.

Copilot uses AI. Check for mistakes.
Comment thread README.md Outdated
| Toast | ✅ | [Docs][Toast Docs] | [2.1.0] | [MIT](LICENSE) |
| Rating | ✅ | [Docs][Rating Docs] | [2.1.0] | [MIT](LICENSE) |
| Slider | ✅ | [Docs][Slider Docs] | [2.0.0] | [MIT](LICENSE) |
| Splitter | ✅ | [Docs][Splitter Docs] | [7.1.0] | [MIT](LICENSE) |
Copy link

Copilot AI Apr 17, 2026

Choose a reason for hiding this comment

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

PR description still contains the placeholder Closes # without an issue/reference. If there isn't a related issue, consider removing that line; otherwise, link the correct issue/PR so the closure automation works as intended.

Copilot uses AI. Check for mistakes.
@rkaraivanov rkaraivanov merged commit 9e666d0 into master Apr 20, 2026
7 checks passed
@rkaraivanov rkaraivanov deleted the radomirchev-add-splitter branch April 20, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants