refactor(theme-classic): little breadcrumbs improvements

[lex111]

Motivation

• turn a last breadcrumb item into a text (see https://usersnap.com/blog/breadcrumbs/ or https://martech.org/breadcrumb-links-good-user-experience-yes/)

Don’t link current page in breadcrumb navigation
The last item in the breadcrumb trail (current user’s location) is optional — if you want to display it, make sure that it’s not clickable or tappable. Since users are already on the page, it does not make any sense to add a link of the current page to the breadcrumb navigation.

• clean up CSS styles -- background for breadcrumb item is supposed to apply only to active state (hover or current item), now it looks weird, so I don't see the point in changing that default design.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Preview

Related PRs

[lex111]

Is it dead code right now, or should the breadcrumb items size be bigger on desktop?

[slorber]

🙈 I don't remember, but that looks fine to remove it 👍

[netlify]

✔️ [V2]

🔨 Explore the source changes: 679f91b

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/62336f7ddc415c0008a14e65

😎 Browse the preview: https://deploy-preview-6932--docusaurus-2.netlify.app

[github-actions]

⚡️ Lighthouse report for the changes in this PR:

Category	Score
🟠 Performance	54
🟢 Accessibility	100
🟢 Best practices	92
🟢 SEO	100
🟢 PWA	90

Lighthouse ran on https://deploy-preview-6932--docusaurus-2.netlify.app/

[github-actions]

Size Change: -247 B (0%)

Total Size: 792 kB

Filename	Size	Change
website/build/assets/css/styles.********.css	105 kB	-247 B (0%)

ℹ️ View Unchanged

Filename	Size
website/.docusaurus/globalData.json	49.9 kB
website/build/assets/js/main.********.js	599 kB
website/build/index.html	38.7 kB

_{compressed-size-action}

[slorber]

Before:

After:

I don't have a strong opinion on this, both look fine to me 👍

Wonder if the icon size change was on purpose? Maybe it's a bit too small now?

---

Also wondering if the :hover effect on the breadcrumb item shouldn't be disabled if the category has no link?

cf:

https://deploy-preview-6932--docusaurus-2.netlify.app/docs/migration/automated/

[slorber]

🙈 I don't remember, but that looks fine to remove it 👍

[lex111]

Wonder if the icon size change was on purpose? Maybe it's a bit too small now?

I would use 0.8 multiplier, so yes, maybe worth to increase by 0.1 for all devices.

Also wondering if the :hover effect on the breadcrumb item shouldn't be disabled if the category has no link?

Don't think so, the active state should be the same whether item is a link or not.

[slorber]

👍