Skip to content

notification-badge-navigation - #912

Open
Edem-Ag wants to merge 6 commits into
mainfrom
notification-badge-navigation
Open

notification-badge-navigation#912
Edem-Ag wants to merge 6 commits into
mainfrom
notification-badge-navigation

Conversation

@Edem-Ag

@Edem-Ag Edem-Ag commented Oct 17, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@Edem-Ag
Edem-Ag requested a review from Jon-Rowe-HMRC October 17, 2024 14:33

@Jon-Rowe-HMRC Jon-Rowe-HMRC left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'The notification badge is a number displayed to the right-hand side of the service navigation.' > 'The notification badge is a number displayed to the right-hand side of the service navigation link it refers to.'

Also, the example link needs to have the correct class (see service navigation component).

@Edem-Ag Edem-Ag left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just realised I have 2 classes on 10

@Jon-Rowe-HMRC

Jon-Rowe-HMRC commented Oct 18, 2024

Copy link
Copy Markdown
Contributor

Needs to use the link classes from https://design-system.service.gov.uk/components/service-navigation/

@Edem-Ag
Edem-Ag requested a review from timsb October 18, 2024 08:39
@Edem-Ag

Edem-Ag commented Oct 18, 2024

Copy link
Copy Markdown
Contributor Author

Hey @timsb

We've used a service navigation class in the example. Over to you to integrate the GOVUK css?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants