[PR #12046] refactor: replace div with button for better accessibility #27477

Closed
opened 2026-02-21 20:41:36 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/12046

State: closed
Merged: Yes


This pull request includes several changes to the CreateAppCard component in the web/app/(commonLayout)/apps/NewAppCard.tsx file. The main changes involve replacing div elements with button elements for better accessibility and adding a display name to the CreateAppCard component.

Accessibility improvements:

  • Replaced div elements with button elements to ensure proper semantics and accessibility for interactive elements.

Code quality improvements:

  • Added CreateAppCard.displayName to improve debugging and component identification in development tools.
  • Removed an unnecessary eslint-disable-next-line comment that was no longer needed.
**Original Pull Request:** https://github.com/langgenius/dify/pull/12046 **State:** closed **Merged:** Yes --- This pull request includes several changes to the `CreateAppCard` component in the `web/app/(commonLayout)/apps/NewAppCard.tsx` file. The main changes involve replacing `div` elements with `button` elements for better accessibility and adding a display name to the `CreateAppCard` component. Accessibility improvements: * Replaced `div` elements with `button` elements to ensure proper semantics and accessibility for interactive elements. Code quality improvements: * Added `CreateAppCard.displayName` to improve debugging and component identification in development tools. * Removed an unnecessary `eslint-disable-next-line` comment that was no longer needed.
yindo added the pull-request label 2026-02-21 20:41:36 -05:00
yindo closed this issue 2026-02-21 20:41:36 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#27477