Skip to content

Commit

Permalink
Fix var name (#151)
Browse files Browse the repository at this point in the history
* Update `msgBanner` to accept JSX and update documentation.  Includes optional positioning above or below the header.

* add warning for developers using msgBannerPosition

* fix var name

* Add Progress Load Bar Component (#143)

* Create progress component and add import for lib

* Create documentation page. Add Route + Link.

* Ensure width is a whole % to avoid css issues

* 3.11.0

* expose defaultIndex on Tabs component (#148)

* expose defaultIndex on Tabs component

* add defaultIndex prop documentation

* 3.12.0

* Link Provider (#142)

* Add LinkProvider and Link components

* Add hrefMap prop to LinkProvider

Allows user to map a different prop key to the default href key so that
internal groundwork usage still works, e.g. for React Router
hrefMap="to"

* Replace existing catalyst Links with gw version

* Put the catalyst Link component back in its box

* Update library internal links to use <Link>

External links can remain as <a>

* Forward ref in Link component

Need to forward ref for Link to maintain compatibility with the
redux-bundler routing pattern.

* Export Link and LinkProvider with library

* Add documentation for LinkProvider

* Remove unnecessary statement

* Add docs example demonstrating ClientRouter usage

* 3.13.0

* 3.14.0

---------

Co-authored-by: Charles Graham <charles.r.graham@usace.army.mil>
Co-authored-by: Brandon Kolze <james.b.kolze@usace.army.mil>
  • Loading branch information
3 people authored Feb 5, 2025
1 parent 1eacd49 commit 1db3043
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/composite/site-wrapper/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ function SiteWrapper({
warningTimeout = undefined,
}) {
// Warn developer that msgBannerPosition should either be top or bottom, but nothing else.
if (!["top", "bottom"].includes(messageBannerPosition)) {
if (!["top", "bottom"].includes(msgBannerPosition)) {
console.warn(
"You must specify top or bottom for messageBannerPosition. Ignoring parameter"
);
Expand Down

0 comments on commit 1db3043

Please sign in to comment.