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

Bug 1718776 - Back/Forward navigation does not work between Ublock Origin option pages #1648

Closed
1 task
alice0775 opened this issue Jul 5, 2021 · 3 comments
Closed
1 task
Labels
bug Something isn't working fixed issue has been addressed

Comments

@alice0775
Copy link

Prerequisites

  • [✔ ] I verified that this is not a filter issue
  • [ ✔] This is not a support issue or a question
  • I performed a cursory search of the issue tracker to avoid opening a duplicate issue
    • Your issue may already be reported
  • [✔ ] The issue is not present after wholly disabling uBlock Origin ("uBO") in the browser
    • If the issue is still present after wholly disabling uBO in the browser, then the issue is unrelated to uBO
  • I tried to reproduce the issue when...
    • [✔ ] uBO is the only extension
    • [✔ ] uBO with default lists/settings
    • [ ✔] using a new, unmodified browser profile
  • [✔ ] I checked the documentation to understand that the issue I report is not a normal behavior

Description

[Description of the bug or feature]
Back/Forward navigation does not work between Ublock Origin option pages.
At first I thought this was a bug in Firefox, but the mozilla developers pointed out that it was a bug on the add-on side. See https://bugzilla.mozilla.org/show_bug.cgi?id=1718776#c2

A specific URL where the issue occurs

[A specific URL is MANDATORY for issue happening on a web page, even if it happens "everywhere"]
Ublock Origin option pages

Steps to Reproduce

  1. install Ublock Origin https://addons.mozilla.org/en-US/firefox/addon/ublock-origin/
  2. Open dashboard (Options)
  3. Navigate Settings, Filter lists and My filters, etc.
  4. Try Back/Forward navigation

Expected behavior:

[What you expected to happen]
Back/Forward navigation should always be working.
And Addressbar and dashboard should be synced.

Actual behavior:

[What actually happened]
Back/Forward navigation is not working some times.
And Addressbar and dashboard do not match.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1718776#c2

Your environment

  • uBlock Origin version: 1.36.0
  • Browser Name and version: Firefox89.0.2
  • Operating System and version: Windows10 Home 21H1 19043.1052
@gwarser gwarser added the bug Something isn't working label Jul 5, 2021
@gorhill
Copy link
Member

gorhill commented Jul 10, 2021

Note that I use self.location.replace() to change the URL in the address bar, my understanding is that this should not add an entry in the history stack, but the behavior I am seeing is that there seems to be one entry entries added -- this occurs in both Firefox and Chromium.

@gorhill
Copy link
Member

gorhill commented Jul 10, 2021

Changing the src attribute on the embedded iframe is what causes a history entry to be created.

gorhill added a commit to gorhill/uBlock that referenced this issue Jul 10, 2021
@gorhill
Copy link
Member

gorhill commented Jul 10, 2021

The fix is to not allow going back and forth, so the issue was that this was possible at all. The reason to not allow going back and forth through browser history is that uBO must check whether there are pending changes in the current pane, and if so the user is warned about this, and must choose whether to keep or loose the changes.

@gwarser gwarser added the fixed issue has been addressed label Jul 10, 2021
@gwarser gwarser closed this as completed Jul 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed issue has been addressed
Projects
None yet
Development

No branches or pull requests

3 participants