remix-run / react-router-website Goto Github PK

Describe what's incorrect/missing in the documentation

Example: https://reactrouter.com//google.com/
Expectation: 404
Actual: 302 redirects to google.com

This is bad because a malicious actor can send links that look like official reactrouter.com pages but the end users get silently redirected to a possible malicious site owned by the sender

I haven't dug too much into this yet but also see that it affects the main remix.run site too - https://remix.run//google.com/

I was receiving an error stating "Unexpected use of 'confirm'"

I was able to resolve this by adding window.confirm instead of just confirm

What version of React Router are you using?

6.4.3

Steps to Reproduce

• Create a simple react project with react-router 6.4.3
• Call the useLocation hook -> const location = useLocation()
• Hover the useLocation() and a pop-up will appear:

Expected Behavior

Redirecting to the useLocation() docs page

Actual Behavior

Redirecting to 404 page

Edit: All the hooks docs url are wrong. The problem here is that i'm using an old react router version (6.4.3) instead of the main one (6.5.0)

It's complaining about the node:child_process, "Error: Cannot find module 'node:child_process'".

Adding search to reactrouter.com would significantly reduce time spent looking for the correct function / component

Description:

I'd like to propose a feature for the official React Router documentation website. While going through the various examples in the documentation, I noticed that there isn't a straightforward way to copy the code samples.

Feature Description:

It would be extremely helpful if a 'Copy' button could be added to each of the code blocks within the documentation. This button would allow users to instantly copy the code samples to their clipboard, reducing the chance of copying errors and saving time.

Benefits:

• Increased efficiency: This feature would allow developers to easily copy and paste code into their IDE without needing to manually select and copy the code, saving time and effort.
• Reduced error potential: By enabling one-click copying, we can minimize the chance of copying errors that might occur when users manually select code blocks.
• Improved user experience: This feature would make the documentation more user-friendly and increase its utility for developers of all experience levels.

I believe this would be a valuable addition to the React Router documentation, and I'd be happy to discuss it further if needed.

What version of React Router are you using?

docs

Steps to Reproduce

1. navigate to https://reactrouter.com/
2. click on any link
3. click on go back (browser)

Expected Behavior

should not crash

Actual Behavior

Happens not only to me.

          We have an example for this: https://github.com/remix-run/react-router/tree/dev/examples/auth

Originally posted by @timdorr in remix-run/react-router#10206 (comment)

Sorry, not shure is there a bug or stackblitz fault, but

$ vite
failed to load config from /home/projects/mkligxplv.github/vite.config.ts
error when starting dev server:
Error: The package "@esbuild/linux-x64" could not be found, and is needed by esbuild.

on https://stackblitz.com/github/remix-run/react-router/tree/main/examples/auth?file=src%2FApp.tsx

In this case, the nested route on the list on the right should be either highlighted or underlined.

I have yet to try Remix, but this would be an excellent PR to get my feet wet.

Is anyone working on this?

http://www.reactrouter.com and https://www.reactrouter.com currently show a 500 error.
Should redirect to https://reactrouter.com instead.

Describe what's incorrect/missing in the documentation

I noticed some misbehavior in the functionality when changing themes in React Router Docs.
The theme is changing back or back and forth between dark/light when you:

1. click on the theme icon and change the theme and then immediately click on the h2 (for example in the Tutorial page heading with the id="data-writes--html-forms") or
2. clicking on the empty space between the main text and the sidebar with the headings

This is checked on both Chrome (Version 118.0.5993.117 ) and Firefox (Version 118.0.5993.117 )