Skip to content

Add tag feature and refactor UI components#149

Merged
RishadAlam merged 6 commits intomainfrom
feat/tags
Apr 13, 2026
Merged

Add tag feature and refactor UI components#149
RishadAlam merged 6 commits intomainfrom
feat/tags

Conversation

@RishadAlam
Copy link
Copy Markdown
Member

This pull request introduces backend and frontend changes to support integration tag management and significantly improves the table UI in the frontend. The backend adds a new controller and routes for managing integration tags, while the frontend enhances the table component with a modernized bulk actions menu, improved column visibility controls, and custom icons for a better user experience.

Backend: Integration Tag Management

  • Added a new IntegrationTagController with get and save methods for retrieving and updating integration tags, including permission checks and data normalization.
  • Registered new AJAX routes for fetching and saving integration tags in ajax.php.
  • Imported the new controller in the route definitions.

Frontend: Table UI Enhancements

  • Redesigned the bulk actions menu in the Table component to use a dropdown with custom icons, supporting bulk tag assignment, delete, status change, and duplication, with improved state handling for menu visibility. [1] [2] [3] [4]
  • Updated the column visibility menu to use a new icon, custom button styling, and removed reliance on the Scrollbars library for a simpler, more consistent UI. [1] [2]
  • Added custom SVG icon components (VerticalDotsIcn, HorizontalDotsIcn, TagAssignIcn) for use in menus and actions, improving visual clarity and accessibility. [1] [2]
  • Improved the global search filter to accept a custom placeholder and optimized event handling. [1] [2]

Other UI Improvements

  • Refactored the Menu component for better tooltip, style, and event handling, making it more reusable and robust. [1] [2]
  • Cleaned up and standardized button and menu class names across components for consistency. [1] [2]

These changes lay the groundwork for integration tag features and provide a more intuitive and maintainable table interface for users and developers.

Copilot AI review requested due to automatic review settings April 13, 2026 07:00
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 13, 2026

✅ WordPress Plugin Check Report

✅ Status: Passed

📊 Report

All checks passed! No errors or warnings found.


🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a tagging system for integrations, allowing users to categorize, filter, and bulk-manage their workflows. The changes include a new backend controller and AJAX routes for tag persistence, alongside significant frontend updates to the table component and integrations page for a more robust management experience. A medium severity issue was identified in the IntegrationTagController where the save method lacks input validation, which could result in accidental data loss.

Copy link
Copy Markdown

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 adds integration tag management (backend AJAX endpoints + frontend tag UI) and refactors the shared table UI to modernize bulk actions, column visibility controls, and icon usage.

Changes:

  • Backend: introduce IntegrationTagController and register new AJAX routes to get/save integration tag data.
  • Frontend: add tag filtering/assignment UX to the Integrations list, including tag modals and per-row/bulk tag assignment.
  • Frontend: refactor the Table/Menu components (bulk actions dropdown, column visibility menu, search placeholder support, updated icons/styles).

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
frontend/src/resource/sass/app.scss Adds styling for tag UI/modals and updates table/checkbox styling.
frontend/src/pages/AllIntegrations.jsx Implements tag CRUD/filtering/assignment flows and wires bulk tag assignment into the table.
frontend/src/components/Utilities/TableCheckBox.jsx Updates checkbox SVG structure (circle + check/indeterminate glyphs).
frontend/src/components/Utilities/Table.jsx Refactors top controls layout, adds bulk actions dropdown, adds search placeholder support, updates column visibility menu UI.
frontend/src/components/Utilities/MenuBtn.jsx Updates row action menu button UI to use a custom dots icon and new class names.
frontend/src/components/Utilities/Menu.jsx Refactors reusable menu behavior and styling/tooltip options.
backend/Routes/ajax.php Registers AJAX routes for integration tag get/save.
backend/controller/IntegrationTagController.php Adds controller to permission-check, normalize, and persist integration tag data.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings April 13, 2026 10:31
Copy link
Copy Markdown

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

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RishadAlam RishadAlam merged commit ae67075 into main Apr 13, 2026
1 check passed
@RishadAlam RishadAlam deleted the feat/tags branch April 13, 2026 11:37
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