Skip to content

Rename Navs to Navs and Tabs#31965

Merged
patrickhlauke merged 7 commits into
mainfrom
navs-tabs
Oct 25, 2020
Merged

Rename Navs to Navs and Tabs#31965
patrickhlauke merged 7 commits into
mainfrom
navs-tabs

Conversation

@mdo
Copy link
Copy Markdown
Member

@mdo mdo commented Oct 25, 2020

@patrickhlauke
Copy link
Copy Markdown
Member

hah, had that issue of mine on my perennial "need to really do a PR when i find a minute..." pile. thanks, will review now.

@patrickhlauke
Copy link
Copy Markdown
Member

build fails. i believe you also need to change the title: in the renamed navs-tabs.md file

@patrickhlauke
Copy link
Copy Markdown
Member

patrickhlauke commented Oct 25, 2020

i believe it also needs changing in the overview.md file for components (see the changes I had to make back when i renamed checks) 490acf6

edit: ah, i'm clearly talking rubbish since there's no overview for components...but yeah there's "some" other reference somewhere that makes this fail, hmmm...

turns out, it was caused by other pages referring to the navs page which has now been renamed and was causing broken links

as we have a precedent of "Checks and radios" for not capitalising beyond the first letter
@patrickhlauke patrickhlauke merged commit 1b6caf6 into main Oct 25, 2020
@patrickhlauke patrickhlauke deleted the navs-tabs branch October 25, 2020 20:30
title: Navs and tabs
description: Documentation and examples for how to use Bootstrap's included navigation components.
group: components
aliases: "/docs/5.0/components/navs/"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we need this alias since v5 is a major version. I need to remember to drop any redundant redirects when we hist stable.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

might still be nice to keep around in case people come from older versions and just hacked the url with 5.0? not precious about it either way though

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We always followed this pattern when we released major/minor versions since the URLs change. I'd rather keep things clean because we can easily end up with hundreds of redirects.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename "Navs" in documentation to "Navs and tabs" or create separate "Dynamic tabs" document

3 participants