Skip to content

Conversation

zamoore
Copy link
Contributor

@zamoore zamoore commented Sep 19, 2025

📌 Summary

If merged, this PR removes the @isVisuallyHidden argument from the HdsAdvancedTableTh component.

🛠️ Detailed description

The functionality for controlling whether the label of the component is visually hidden is done through the column definition rather than by passing an argument to the component.

This change updates the documented functionality without actually changing how the component works.

🔗 External links

Jira ticket: HDS-5293


👀 Component checklist

💬 Please consider using conventional comments when reviewing this PR.

📋 PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

@zamoore zamoore requested review from a team as code owners September 19, 2025 18:45
@hashibot-hds hashibot-hds added packages/components docs-website Content updates to the documentation website labels Sep 19, 2025
@zamoore zamoore force-pushed the zamoore/hds-4293/AdvancedTable-remove-isVisuallyHidden branch from ce97b10 to dbeaa24 Compare September 19, 2025 18:47
Copy link

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
hds-showcase Ready Ready Preview Sep 19, 2025 6:51pm
hds-website Ready Ready Preview Sep 19, 2025 6:51pm

@zamoore zamoore merged commit 7c0bc36 into main-5.0.0 Sep 19, 2025
15 of 16 checks passed
@zamoore zamoore deleted the zamoore/hds-4293/AdvancedTable-remove-isVisuallyHidden branch September 19, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-website Content updates to the documentation website packages/components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants