Skip to content
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

Missing TH on admin form tables #2941

Open
rachidatecs opened this issue Oct 15, 2024 · 1 comment
Open

Missing TH on admin form tables #2941

rachidatecs opened this issue Oct 15, 2024 · 1 comment
Labels
accessibility bug Something that isn't working as intended Feature: 🧮 Analyst Experience

Comments

@rachidatecs
Copy link
Contributor

Current Behavior

The custom tables we're using throughout DJA for contacts feature a 'copy email' column on the right. That column 's TDs do not have any associated TH.

Expected Behavior

Every TD should have an associated TH either in the THEAD or as the first table cell in its row.

Per this slack discussion, we should have a visible TH that says ‘Action’

This should be applied to every instance of DJA table that features a 'copy email' button.

Steps to Reproduce

  1. Navigate to a domain request with other contacts or portfolio with members and admins.
  2. Inspect table.

Environment

No response

Additional Context

No response

Issue Links

No response

@rachidatecs rachidatecs added accessibility bug Something that isn't working as intended labels Oct 15, 2024
@abroddrick
Copy link
Contributor

abroddrick commented Oct 15, 2024

Soft sprinting for 57, just because it's a nice easy ticket that shouldn't take a ton of time. Adding analyst experience, though it's mostly an accessibility thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug Something that isn't working as intended Feature: 🧮 Analyst Experience
Projects
Status: 🎯 Ready
Development

No branches or pull requests

2 participants