-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug - ClipboardCopy - ClipboardCopyToggle has a div within a button #9778
Comments
Also a followup to comment chain #9574 (comment) We may not want to spread the className to the div internal to ClipboardCopyToggle as we don't seem to really do that for other _Toggle components, e.g. AccordionToggle. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This issue has been automatically marked as stale because it has not had recent activity. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This issue has been closed because it has not had activity since being marked as stale. |
Followup to patternfly/patternfly#6019
Describe the problem
ClipboardCopy's expansion toggle currently has a div within a button, which is invalid markup.
Expected behavior
The
pf-v5-c-clipboard-copy__toggle-icon
element should be a span rather than a div.The text was updated successfully, but these errors were encountered: