Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

InternalHeader to include NavigationLinks #5

Closed
frobian21 opened this issue Jan 27, 2022 · 2 comments
Closed

InternalHeader to include NavigationLinks #5

frobian21 opened this issue Jan 27, 2022 · 2 comments

Comments

@frobian21
Copy link

Related component

InternalHeader

Context

We would like to use navigation items with an InternalHeader like the ones that are available for Header, which is currently not available.

Alternatives

Currently, we've manually adjusted the HTML instead of using the InternalHeader component, we had to add the data module so that the javascript was pulled in.

Additional information (if applicable)

None, it's an internal component so we didn't do accessibility testing - Besides, I imagine the Navigation Links have already been tested with the standard Header

@oscarduignan
Copy link

HI @frobian21 thank you for sharing this, we're going to raise the possibility of changing the internalHeader component API with the design resources team, and when we have a ticket to track any work related I'll share it with you directly 👍

In terms of accessibility testing, I'm not familiar enough with the specifics of how the navigation links are coded to say for sure that use as they are within the internal header would not cause any accessibility issues, so I would always advise retesting. With accessibility it's also especially important to understand that the accessibility of a use of a component is context dependant, so an accessible pattern could be used in an inaccessible way, depending on the context.

I think the data-module bit you had to adjust is a long standing (but unknown to us) bug that we should soon have fixed with the hmrcHeader component hmrc/hmrc-frontend#207

@oscarduignan
Copy link

oscarduignan commented Feb 22, 2022

Hi @frobian21 we were able to raise this with the design resources working group last week. The consensus was that rather than adding menu items to the internal header, they would suggest using a pattern like the Personal Tax Account (PTA) Account Menu component (HmrcAccountMenu). Which isn't provided as a generic component at the moment, it's PTA specific, but it seems like based on the demand for a generic version of something like this, that providing one is probably on the future roadmap for design resources. To follow any progress on that / contribute to the design you can comment in and subscribe to the design pattern issue thread for the account header pattern

CleanShot 2022-02-22 at 07 53 38@2x

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants