grssam / UIEnhancer

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

Feedback (cont. of #16) #19

Open ghost opened 12 years ago

ghost commented 12 years ago

Was a bit occupied these days - could not answer, sry.

http://db.tt/SbhSrPdg

1) Yes, screen №4 is from another addon, but i think (as a feature request) this way is better - to see both status and URL in different placeholders.

2) Screen №6. A long URL does not take all the space in location bar. 3) Feature request Long URLs should be cropped in the middle - to see both the domain and the ending at the same time. 4) If you scroll a long URL, then the left Scroll Arrow is not dissabled (screen №7).

5) Feature request Clicking middle button on any part of a breadcrumb should behave like a simple link (by default open in a new tab).

6) About the 'y'. Just got what u mean. There is also such a letter that is a consonant, but in your casse should be regarded as a vowel - that is йЙ . 7) Gibberish removal behaves strangely on such sites like Youtube.

grssam commented 12 years ago

1) It is a bit difficult, as when there is some status showing up and you then hover over a link, the status bar itself drops the status and displays the link's address. I don't knw how to reproduce the status while Firefox itself is not displaying it. Still I will look ifurther into it.

2) Some space is left intentionally to allow clicking on the blank area to have the normal url in textual form. 3) Then I need to think of a good and usefull UI for that case, maybe you can help and give some input for the design. 4) Will do. 5) Yes, it does that, but only till the part clicked. 7) In some cases when the video address (after v= ) contains '_' , then it does not replaces it with video name, but otherwise, it replaces the random address part, with the video name.

ghost commented 12 years ago

1) I think (if you gona do this, and i hope you will =) ) you can have a look at Link Location Bar addon (https://addons.mozilla.org/ru/firefox/addon/link-location-bar/?src=api).

5) This should also work with the last part of the URL as well as with the links in a dropdown menu.

2) The space can be removed by adding a small icon to the right of the URL if it is too long (like Smart Text addon - https://addons.mozilla.org/ru/firefox/addon/smart-text/?src=search).

3) Got some thoughht on this:

First is - domain should be always visible (maybe except when you intentionally scroll the url). However, showing it in an identity box is not a good idea, as Favicon with domain info should be separated from domain url (maybe at least some minor separator, but still). Then domain should have the same design as the rest of the url (breadcrumbs), cause it IS a part of it. Second (some kind of a feature request) - it should be possible to easily remove domain levels (via a dropdown maybe).

And here's the idea my friend suggested: [ subdomain | domain ] <<... dir5 / dir6 / page ? param1 & param2 ...>> [ # ] The [] part is what should always be seen. << and >> are scrolls. The text between should always (if needed) be centered on PAGE. And the # part may look like a small indicator, that shows the text on hover or alike.

Might also as well add a feature to see all the anchors on a page via the # part.

grssam commented 12 years ago

1) No, no, that is easy, showing the location of the link in the urlbar is easy to program from the start, what I meant was that when you hover over a link while Firefox was displaying some status like Connecting to ... , or waiting for .. , then it shows the link url instead of displaying the status for the time your mouse are over the link. So to generate those status like conneting to ... etc is difficult, and not the generating the location of the url. Still I think I have an idea in mind which I will test soon.

2) Yes I can do that, but a button will occupy around 24px, and what I use is around 100 px blank space. This blank space as compared to the cutton is easier to access. But in your case, i.e. the screenshot you displayed, the extra space is due to the fact that I have to leave a minimum of 250px space for the status when you have checked to show status in Location Bar. Otherwise it uses only 100px.

5) It does, doesn't it ? Please try it, it works where ever possible, in drop downs, on parts, while scrolling on a part etc etc.

3) Nice thought, if you look at it, this is waht exactly happens when you have checked "use identity box to display domain" feature, except for the centering to page part. Centering to page part is a bit debatable as only 50% url's are supposed to work in a way that page is the most important part of the url. But in fact the most important part of a url in the parameters. Like in google search, youtube, facebook, bugzilla and many more, the params is what define the page and each unique page has unique params and not unique 'page' part.

Here's what I think I should do : a) Have the domain part always visible, that means when even when you scroll to right, it will display the domain part. b) The above feature will only work when domains are not displayed in Identity box (either by pref'ing the feature off, or in case of secured sites) c) Have a drop down entry for each domain level.

Adding a button will increase the UI in Location Bar, which I don't really want to do unless important. Since the button's use here would just be to display the url in text form (which can be done by Ctrl + hover, or clicking in the right blank space also), thus for now the button is a no-go.

I didn't get the last feature request you made.

grssam commented 12 years ago

For point 3) I came up with an intermediate solution : Untill possible display the url like this : [ subdomain | domain ] / dir5 / dir6 / page << ...param3 & param4

But when the url till the 'page' part is itself too big, switch to this method [ subdomain | domain ] <<... page ? param1 & param2 & param3 & param4 i.e. displaying the domain, but scrolled up to the end.

This might take a while to complete as it involves a lot of algorithm changes

ghost commented 12 years ago

2) Tried the both options - does not change its behavior. Still i think that adding a little button (like "...") is way better than that space in both appearance and usability (at least it wont loose a thing).

5) Tried. Screens №8-9 http://db.tt/SbhSrPdg . The underlined parts do not react to niddle clicks.

3) The domain should not be placed inside Identity box, as these two elements have different uses so to say.

And i dont think this [ subdomain | domain ] / dir5 / dir6 / page << ...param3 & param4 will be good, as it breaks the consistency of design. Just let it not center on PAGE as long as there are not that many parts to the left (as i do agree that DIR is way more important).

About the # button: This part should be reworked a bit to make it more distinguishable from the rest of the url (as at least the separator should be # , not > ). As i said - it is possible to replace the anchor text (if it does not fit inside the location bar) with just "#" button that shows the anchor on hover/click.

The last feature request: For example, on this page https://github.com/scrapmac/UIEnhancer/issues/19 there are different anchors like

issuecomment-4312852

issuecomment-4311212

that will bring you to the corresponding comment whe you click them. So my idea was to add a list of all of this anchors on the current page in the dropdown list corresponding to the last ( # ) breadcrumb of the url.

grssam commented 12 years ago

5) Because they are the current pages, I do not listen to the links that correspond to the location of the current page, maybe atleast I should allow middle click. 3) Then you can turn that feature off. Have you seen the options? you can customize almost everything in my addon via options window.

If you switch to the new theme, then the anchor part and the query string part are easily distinguishable as they are in different color, and even that is customizable via css stylesheet. I cannot do much change with the old theme specifically.

About the feature request, I will look into it, if there is any asynchronous way to do it, I will do it, otherwise, I do not intend to hamper the web browsing of my users.