-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[docs] Fix Link typings in the react-router example #32308
Conversation
@danilo-leal - it is green now - yeah |
@@ -16,9 +16,15 @@ You can also make a button perform navigation actions. You can also make a butto | |||
|
|||
In real-life applications, using a native `<a>` element is rarely enough. element is rarely enough. You can improve the user experience by using an enhanced Link component systematically. The theme of Material UI allows configuring this component once. For instance, with react-router: The theme of MUI allows configuring this component once. For instance, with react-router: | |||
|
|||
```jsx | |||
```tsx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are not manually changing the translation files. Please revert the changes in the routing-zh.md
and routing-pt.md
and we should be good to merge :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mnajdova addressed
Could you please also update the LinkRouterWithTheme demo to be consistent with the example (type the forwardRef argument)? |
@michaldudak - updated (I erased my previous comment - I misread your comment - meh 😿 ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍
Closes #32306