From b54f0d6bec1b49e57c6a53b989e0638153f2855c Mon Sep 17 00:00:00 2001
From: TJB <tjb042@users.noreply.github.com>
Date: Fri, 5 Apr 2024 19:54:31 -0400
Subject: [PATCH 1/4] Update Navbar.tsx

Add note to `collapseOnSelect` based on feedback in https://github.com/react-bootstrap/react-bootstrap/issues/6654
---
 src/Navbar.tsx | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/Navbar.tsx b/src/Navbar.tsx
index 05680358c5..676e31a93d 100644
--- a/src/Navbar.tsx
+++ b/src/Navbar.tsx
@@ -109,6 +109,9 @@ const propTypes = {
    * Toggles `expanded` to `false` after the onSelect event of a descendant of a
    * child `<Nav>` fires. Does nothing if no `<Nav>` or `<Nav>` descendants exist.
    *
+   * `<Nav.Link>` descentants of `<Nav>` will not trigger the onSelect event unless
+   * the `eventKey` or `href` props are defined.
+   *
    * Manually controlling `expanded` via the onSelect callback is recommended instead,
    * for more complex operations that need to be executed after
    * the `select` event of `<Nav>` descendants.

From b00e9d069b1cac9637944ecd1b3b015555faf393 Mon Sep 17 00:00:00 2001
From: TJB <tjb042@users.noreply.github.com>
Date: Fri, 5 Apr 2024 20:02:28 -0400
Subject: [PATCH 2/4] Update Navbar.tsx

---
 src/Navbar.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Navbar.tsx b/src/Navbar.tsx
index 676e31a93d..bb3708a0a9 100644
--- a/src/Navbar.tsx
+++ b/src/Navbar.tsx
@@ -109,8 +109,8 @@ const propTypes = {
    * Toggles `expanded` to `false` after the onSelect event of a descendant of a
    * child `<Nav>` fires. Does nothing if no `<Nav>` or `<Nav>` descendants exist.
    *
-   * `<Nav.Link>` descentants of `<Nav>` will not trigger the onSelect event unless
-   * the `eventKey` or `href` props are defined.
+   * `<NavLink>` descentants of `<Nav>` will not trigger the onSelect event unless
+   * an `eventKey` or `href` prop is defined.
    *
    * Manually controlling `expanded` via the onSelect callback is recommended instead,
    * for more complex operations that need to be executed after

From 21bb7863db5a8962bcf129e749abea4529ab3f64 Mon Sep 17 00:00:00 2001
From: TJB <tjb042@users.noreply.github.com>
Date: Fri, 5 Apr 2024 20:03:10 -0400
Subject: [PATCH 3/4] Update NavLink.tsx

---
 src/NavLink.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/NavLink.tsx b/src/NavLink.tsx
index 8e70ac08eb..e0a3265155 100644
--- a/src/NavLink.tsx
+++ b/src/NavLink.tsx
@@ -37,12 +37,16 @@ const propTypes = {
    * */
   role: PropTypes.string,
 
-  /** The HTML href attribute for the `NavLink` */
+  /** 
+   * The HTML href attribute for the `NavLink`. Used as the unique identifier
+   * for the `NavLink` if an `eventKey` is not provided.
+   */
   href: PropTypes.string,
 
   /**
    * Uniquely identifies the `NavItem` amongst its siblings,
    * used to determine and control the active state of the parent `Nav`
+   * as well as onSelect behavior of a parent `Navbar`.
    */
   eventKey: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
 

From d4d6cd9e29f7609934aaab758f155ed23a20a55a Mon Sep 17 00:00:00 2001
From: TJB <tjb042@users.noreply.github.com>
Date: Fri, 7 Jun 2024 21:09:22 -0400
Subject: [PATCH 4/4] Update src/Navbar.tsx

Co-authored-by: Kyle Tsang <6854874+kyletsang@users.noreply.github.com>
---
 src/Navbar.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Navbar.tsx b/src/Navbar.tsx
index bb3708a0a9..606b14dc26 100644
--- a/src/Navbar.tsx
+++ b/src/Navbar.tsx
@@ -109,7 +109,7 @@ const propTypes = {
    * Toggles `expanded` to `false` after the onSelect event of a descendant of a
    * child `<Nav>` fires. Does nothing if no `<Nav>` or `<Nav>` descendants exist.
    *
-   * `<NavLink>` descentants of `<Nav>` will not trigger the onSelect event unless
+   * `<NavLink>` descendants of `<Nav>` will not trigger the `onSelect` event unless
    * an `eventKey` or `href` prop is defined.
    *
    * Manually controlling `expanded` via the onSelect callback is recommended instead,