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

Change icon to proxy name/color used for tab url #114

Closed
NewtonChutney opened this issue Jan 8, 2024 · 5 comments
Closed

Change icon to proxy name/color used for tab url #114

NewtonChutney opened this issue Jan 8, 2024 · 5 comments
Labels
area: tab-proxy Tab Proxy related issues done ✓

Comments

@NewtonChutney
Copy link

NewtonChutney commented Jan 8, 2024

If I'm not wrong, the version before the Chrome codebase and Firefox were merged, did update the icon color/name to reflect the proxy used for the page's URL
It appears that is no longer the case.. Also, there's no indicator for the Set Tab proxy function. This could help it

Firefox v121.0
FoxyProxy v8.8 (22 Dec, 2023)

Edit: I check for duplicates but couldn't find any, please do close this in that case

@erosman
Copy link
Collaborator

erosman commented Jan 8, 2024

If I'm not wrong, the version before the Chrome codebase and Firefox were merged, did update the icon color/name to reflect the proxy used for the page's URL

FoxyProxy v3-8 changed the icon badge to show the proxy when a single proxy is selected (for all).
That hasn't changed.

FoxyProxy v7, changed the icon badge when in "Proxy by Patterns" mode as well (Firefox only, not possible on Chrome). However, that was NOT per tab and the same proxy was shown for all the tabs.

FoxyProxy 8, initially dropped the feature but on demand (#57), re-added the feature in v8.7 behind a "Show Pattern Proxy" options.
Additionally, the new feature is per-tab, so the proxy is specific to each tab.

Also, there's no indicator for the Set Tab proxy function.

There is an indicator which is the FoxyProxy icon shown inside the address-bar. On mouse-over, it also shows the details of the proxy involved.

@NewtonChutney
Copy link
Author

FoxyProxy v7, changed the icon badge when in "Proxy by Patterns" mode as well (Firefox only, not possible on Chrome). However, that was NOT per tab and the same proxy was shown for all the tabs.

Ahh, that makes so much sense... Now I understand the random glitches when other tabs load..
Thanks for the fix that made it per-tab

There is an indicator which is the FoxyProxy icon shown inside the address-bar. On mouse-over, it also shows the details of the proxy involved.

Ahh... this indicator..
Untitled
but in case we select show proxy per tab, this creates confusion to the user.. I'd say we override that indicator too?

@erosman
Copy link
Collaborator

erosman commented Jan 8, 2024

but in case we select show proxy per tab, this creates confusion to the user.. I'd say we override that indicator too?

Fair point.... however if the icon badge is going to change, what should it change to?

  1. Clear the badge when in Tab Proxy
  2. Set the badge to match Tab Proxy

I think, the 2nd option might be more versatile (maybe with an additional text in the mouse-over).
This option can mean the address-bar icon can be dropped completely. 🤔

@erosman erosman added the area: tab-proxy Tab Proxy related issues label Jan 8, 2024
@erosman
Copy link
Collaborator

erosman commented Jan 11, 2024

There are some changes in v8.9.

  • Removed "Show Pattern Proxy" option and made it default (from 8.7) (Firefox only)
  • Removed Tab Proxy page-action and set it to the toolbar icon (Firefox only)

@NewtonChutney
Copy link
Author

There are some changes in v8.9.

  • Removed "Show Pattern Proxy" option and made it default (from 8.7) (Firefox only)
  • Removed Tab Proxy page-action and set it to the toolbar icon (Firefox only)

Thanks a lot!! ♥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: tab-proxy Tab Proxy related issues done ✓
Projects
None yet
Development

No branches or pull requests

2 participants