Skip to content

refactor: use Tabler 1.5 tag-list class - #29

Closed
AbdelrhmanSaid wants to merge 1 commit into
masterfrom
tabler-v1.5.1
Closed

AbdelrhmanSaid wants to merge 1 commit into
masterfrom
tabler-v1.5.1

Conversation

@AbdelrhmanSaid

@AbdelrhmanSaid AbdelrhmanSaid commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rename the deprecated Tabler .tags-list container to .tag-list.
  • Keep the change compatible with Tabler 1.4 and 1.5.
  • Make no test changes.

Checks

  • vendor/bin/pint --test src/Datatables/src/Columns/TagsColumn.php

This branch can be consumed temporarily as dev-tabler-v1.5.1 by the dashboard until the PR is merged and a release is tagged.

@greptile-apps

greptile-apps Bot commented Sep 13, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the tags column markup to use Tabler’s tag-list wrapper class while preserving tag escaping and rendering behavior.

  • Renames only the wrapper CSS class.
  • Leaves the generated tag elements and data handling unchanged.
  • No security concerns or repository-rule violations were identified.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete correctness, security, or compatibility failure established.

The change only replaces a static wrapper class; escaping and tag generation remain intact, and repository evidence shows no dependency on the previous wrapper name.

Important Files Changed

Filename Overview
src/Datatables/src/Columns/TagsColumn.php Renames the rendered tags wrapper from tags-list to tag-list without changing content handling.

Reviews (1): Last reviewed commit: "refactor: use Tabler 1.5 tag-list class" | Re-trigger Greptile

@AbdelrhmanSaid
AbdelrhmanSaid deleted the tabler-v1.5.1 branch September 14, 2026 14:25
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.

1 participant