nodejs.org icon indicating copy to clipboard operation
nodejs.org copied to clipboard

Breadcrumbs not updating on Learn pages

Open mikeesto opened this issue 11 months ago • 4 comments

URL:

https://nodejs-bjq0pcoku-openjs.vercel.app/en/learn/getting-started/how-to-install-nodejs

Browser Name:

Chrome

Browser Version:

122.0.6261.69

Operating System:

MacOS

How to reproduce the issue:

The breadcrumbs on the Learn pages display "Diagnostics" regardless of what module the page actually falls under. The breadcrumbs should dynamically update to reflect the correct module. Screen Shot 2024-02-29 at 1 19 49 pm

mikeesto avatar Feb 29 '24 03:02 mikeesto

The breadcrumbs on the Learn pages display "Diagnostics" regardless of what module the page actually falls under. The breadcrumbs should dynamically update to reflect the correct module.

This issue has happened before, and it is usually related to the logic of current pathname, definitely something we should fix.

ovflowd avatar Feb 29 '24 08:02 ovflowd

The breadcrumbs on the Learn pages display "Diagnostics" regardless of what module the page actually falls under. The breadcrumbs should dynamically update to reflect the correct module.

This issue has happened before, and it is usually related to the logic of current pathname, definitely something we should fix.

Is anyone taking it? I have nextjs programming experience and I'm happy to help.

yanggggjie avatar Mar 02 '24 12:03 yanggggjie

Is anyone taking it?

Thanks for your interest. We don't assign tickets if we can help it. Feel free to submit a pull request if you have time. Reference 🍪 https://www.redhat.com/en/blog/dont-lick-cookie

So yeah go ahead

AugustinMauroy avatar Mar 02 '24 12:03 AugustinMauroy

The breadcrumbs on the Learn pages display "Diagnostics" regardless of what module the page actually falls under. The breadcrumbs should dynamically update to reflect the correct module.

This issue has happened before, and it is usually related to the logic of current pathname, definitely something we should fix.

Is anyone taking it? I have nextjs programming experience and I'm happy to help.

Yeah, feel free to just tackle the work :)

ovflowd avatar Mar 02 '24 15:03 ovflowd