Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Select existing "Advanced settings" page if any#265

Merged
JustOff merged 1 commit intogorhill:masterfrom
hawkeye116477:advancedEx
Oct 10, 2020
Merged

Select existing "Advanced settings" page if any#265
JustOff merged 1 commit intogorhill:masterfrom
hawkeye116477:advancedEx

Conversation

@hawkeye116477
Copy link
Contributor

@hawkeye116477 hawkeye116477 commented Oct 9, 2020

Previously when you wanted to open advanced settings and clicked multiple times, then same multiple tabs would open. Now should be only one.

Changes from 1a020ed are semi related (required).

@JustOff
Copy link
Collaborator

JustOff commented Oct 10, 2020

Could you please clarify a little why 1a020ed is required? I don't see any error messages in the console similar to the ones mentioned in the linked issue.

@hawkeye116477
Copy link
Contributor Author

hawkeye116477 commented Oct 10, 2020

I see now that's not needed, only one additional change in previous commit is needed. I'll change that.

Update: Changed.

Related issue:
- uBlockOrigin/uBlock-issues#591

Additionally, I added a link to the logger in the
"About" pane in the dashboard in order to be able
to access the logger without having to go through
the popup panel.
@JustOff
Copy link
Collaborator

JustOff commented Oct 10, 2020

Thank you!

@JustOff JustOff merged commit fc0b3a5 into gorhill:master Oct 10, 2020
@JustOff JustOff mentioned this pull request Oct 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants