search

grssam / UIEnhancer

Source code for UI Enhancer addon
https://addons.mozilla.org/en-US/firefox/addon/ui-enhancer/
51 stars 22 forks source link

Do not crop HTTP get parameters if not necessary #80

Open wagnerand opened 11 years ago

wagnerand commented 11 years ago

See http://img24.imageshack.us/img24/6420/screenshot2801201309212.png

Original link: http://i.americanapparel.net/storefront/photos/fullscreen.html?l=1&i=http://i.americanapparel.net/storefront/photos/zoom/serve.asp?media=trt497_tri-coffee.jpg

grssam commented 11 years ago

If I am not mistaken, all the three get parameters are in the screenshot .. l, i and media in green color.

wagnerand commented 11 years ago

Right and the value of i is cropped...

grssam commented 11 years ago

Ah, got it.

atomGit commented 11 years ago

personally, i would rather never have the URI changed - i think it should always remain exactly as it is, unless the option to remove gibberish is enabled, but even then the original should be displayed when clicking the address bar (off of the URI)

grssam commented 11 years ago

@atomGit , When remove gibberish is enabled, and you click on the empty part of the location bar (to the right), the url is displayed in its original form only. At that point, I have no control over the url, I simply hide my breadcrumbs then.

atomGit commented 11 years ago

don't have your ext. installed right now, but i swore that when i loaded the example URL...

http://i.americanapparel.net/storefront/photos/fullscreen.html?l=1&i=http://i.americanapparel.net/storefront/photos/zoom/serve.asp?media=trt497_tri-coffee.jpg

...and then clicked off of it to reveal the original, the content of the 'i=' parm was still missing

i could be crazy though :)

grssam commented 11 years ago

You could install the extension once more just to check . Its restart-less so will not cause you a single pain.