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

Remove anchor in URL so avoid browser scrolling to the top of the page when editing order #6736

Merged

Conversation

jibees
Copy link
Contributor

@jibees jibees commented Jan 25, 2021

What? Why?

Closes #6707
Do not add '#' in URL so the browser will not scroll to the top of the page.

What should we test?

  • As an admin, edit order
  • Try to edit the shipping method or tracking info of that order
  • URL must be the same and no jump into the top of the page should happen

Release notes

Avoid unintentional scroll to top when editing an order.
Changelog Category: User facing changes

Adding '#' to the current url makes most of browser jump to the top of the page. Avoid this by deleting this added # (meaningless) and replacing it by empty string.
Copy link
Contributor

@Matt-Yorkley Matt-Yorkley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@filipefurtad0 filipefurtad0 self-assigned this Jan 28, 2021
@filipefurtad0 filipefurtad0 added the pr-staged-fr staging.coopcircuits.fr label Jan 28, 2021
@filipefurtad0
Copy link
Contributor

Hey @jibees ,

Awesome!!

Peek 2021-01-28 19-18

Ready to go.

@filipefurtad0 filipefurtad0 removed the pr-staged-fr staging.coopcircuits.fr label Jan 28, 2021
@sauloperez sauloperez merged commit 0a1d487 into openfoodfoundation:master Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Desktop and Mobile: "jumps" in the UI, when editing shipping methods/tracking info (Backoffice orders)
4 participants