-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Options set via addon should not override URL params #1690
Comments
@Hypnosphi it happens because we use At least I don't think that it's a bug, or I missed something? |
My point is that query params should have higher priority when present |
I see but |
Well, if I reload the page, I stay on the same story. Links to other stories can be adjusted according to their specific |
So you offer to detect where it runs: inside a story or not? And depending on that change |
I'd suggest roughly that:
|
By the way, how do you use |
or even like this: so in this cases URL could be formed twice:
|
Oh, so it's inside story render. Ok, I think my suggestion may be restricted to |
Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 60 days. Thanks! |
I think it’s still an issue worth fixing |
Will fix it as part of #2313 |
yarn start
in repo root⌘ ⇧ D
Expected: Actions panel stays hidden
Actual: It is hidden for a moment, and then shows up
The text was updated successfully, but these errors were encountered: