Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix var name #151

Merged
merged 11 commits into from
Feb 5, 2025
Merged

Fix var name #151

merged 11 commits into from
Feb 5, 2025

Conversation

willbreitkreutz
Copy link
Contributor

No description provided.

willbreitkreutz and others added 11 commits January 27, 2025 11:10
…ptional positioning above or below the header.
* Create progress component and add import for lib

* Create documentation page. Add Route + Link.

* Ensure width is a whole % to avoid css issues
* expose defaultIndex on Tabs component

* add defaultIndex prop documentation
* 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
@willbreitkreutz willbreitkreutz merged commit 1db3043 into main Feb 5, 2025
1 check passed
@willbreitkreutz willbreitkreutz deleted the fix-var-name branch February 5, 2025 17:33
@willbreitkreutz willbreitkreutz removed the request for review from krowvin February 5, 2025 17:33
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.

3 participants