#3499 closed defect (fixed)
Easy way to clear the url entry
Reported by: | erikos | Owned by: | humitos |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Browse | Version: | Git as of bugdate |
Severity: | Minor | Keywords: | patch |
Cc: | garycmartin, manuq | Distribution/OS: | OLPC |
Bug Status: | New |
Description
In Browse I was looking for an easy way to clear the url entry, so that I can type a new url. In iOS you have a button at the right to do this (screenshot attached). The 'x' button is slightly different than the 'x' for stop loading of a page.
Attachments (8)
Change History (28)
Changed 12 years ago by erikos
comment:1 Changed 12 years ago by erikos
- Cc manuq added
comment:2 Changed 12 years ago by callkalpa
So we need two separate buttons. One to stop loading and the other to clear the URL box.
comment:3 follow-up: ↓ 4 Changed 12 years ago by garycmartin
Just to clarify, the option to clear the url field should only appear when the url input has focus (e.g. you click/tap it to efit, or issue the keyboard shortcut ctrl-L). In the place where the little stop page loading icon would be, should be our standard clear field icon (grey circle with while x inside). Once you press the clear field icon, the field should be blanked and the clear field icon disappear (only to reappear once you start typing some text).
comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 12 years ago by erikos
Replying to garycmartin:
Just to clarify, the option to clear the url field should only appear when the url input has focus (e.g. you click/tap it to efit, or issue the keyboard shortcut ctrl-L). In the place where the little stop page loading icon would be, should be our standard clear field icon (grey circle with while x inside). Once you press the clear field icon, the field should be blanked and the clear field icon disappear (only to reappear once you start typing some text).
Correct, that would be the desired behavior in my opinion. And the general behavior is good with you? Should we use similar icons as on iOS (see screenshots)?
comment:5 in reply to: ↑ 4 ; follow-up: ↓ 8 Changed 12 years ago by garycmartin
Replying to erikos:
Replying to garycmartin:
Just to clarify, the option to clear the url field should only appear when the url input has focus (e.g. you click/tap it to efit, or issue the keyboard shortcut ctrl-L). In the place where the little stop page loading icon would be, should be our standard clear field icon (grey circle with while x inside). Once you press the clear field icon, the field should be blanked and the clear field icon disappear (only to reappear once you start typing some text).
Correct, that would be the desired behavior in my opinion. And the general behavior is good with you? Should we use similar icons as on iOS (see screenshots)?
Yes, and the icons are already very similar to the ones iOS used - my only suggestion would be that we consider a square stop icon, to stop a loading page, rather than the current grey x (which we use to close tabs and close control panels).
comment:6 Changed 12 years ago by manuq
I attached a proposal for a stop loading button icon with a black square background. If Gary approves, there is a patch sent to the list.
comment:7 Changed 12 years ago by manuq
Also see old enhacement proposal in #1511 .
comment:8 in reply to: ↑ 5 Changed 12 years ago by erikos
Replying to garycmartin:
Replying to erikos:
Replying to garycmartin:
Just to clarify, the option to clear the url field should only appear when the url input has focus (e.g. you click/tap it to efit, or issue the keyboard shortcut ctrl-L). In the place where the little stop page loading icon would be, should be our standard clear field icon (grey circle with while x inside). Once you press the clear field icon, the field should be blanked and the clear field icon disappear (only to reappear once you start typing some text).
Correct, that would be the desired behavior in my opinion. And the general behavior is good with you? Should we use similar icons as on iOS (see screenshots)?
Yes, and the icons are already very similar to the ones iOS used - my only suggestion would be that we consider a square stop icon, to stop a loading page, rather than the current grey x (which we use to close tabs and close control panels).
Hmm, if you change the stop icon you would need to change the reload one as well, otherwise it looks strange. To me the simple 'x' and 'reload' icon do match nicely together.
As you say, our 'dialog-cancel' icon is already the same as the one in iOS, we can keep that. See #3385 though, for an issue adding that to an entry.
Changed 12 years ago by manuq
comment:9 Changed 12 years ago by manuq
Attached a patch that adds the square background to both icons.
Changed 12 years ago by manuq
comment:10 follow-up: ↓ 11 Changed 12 years ago by garycmartin
After testing the square based options for browse-dialogue-cancel and browse-view-refresh icons I'd like to reject my suggestion we consider a square stop icon (aka browse-dialogue-cancel). The existing stroke based browse icons are the least intrusive (as they make best use of the space without feeling cramped), and we should use sugar-artwork dialogue-cancel for clearing the url entry (or its equivalent icon fixing the wrong size bug #3385).
comment:11 in reply to: ↑ 10 Changed 12 years ago by manuq
Replying to garycmartin:
After testing the square based options for browse-dialogue-cancel and browse-view-refresh icons I'd like to reject my suggestion we consider a square stop icon (aka browse-dialogue-cancel). The existing stroke based browse icons are the least intrusive (as they make best use of the space without feeling cramped), and we should use sugar-artwork dialogue-cancel for clearing the url entry (or its equivalent icon fixing the wrong size bug #3385).
Yeah, fine for me too.
comment:12 Changed 12 years ago by humitos
- Priority changed from Unspecified by Maintainer to Normal
comment:13 Changed 12 years ago by humitos
- Owner changed from erikos to humitos
- Status changed from new to accepted
Changed 12 years ago by humitos
comment:14 Changed 12 years ago by humitos
I've just attached a patch with this behaviour. Although I'm not happy with the icon to clear the url.
Manuq and Gary, can you take a look at my patch and suggest a new icon?
comment:15 Changed 12 years ago by humitos
- Keywords patch added
comment:16 Changed 12 years ago by manuq
Works great humitos. Only exception should be the tabs displaying PDF. They have not editable entries, so the button to clear text should not be displayed for them.
comment:17 Changed 12 years ago by manuq
Also we should use the 'dialog-cancel' icon as per Gary comment 11.
comment:18 follow-up: ↓ 19 Changed 12 years ago by manuq
- Resolution set to fixed
- Status changed from accepted to closed
Found this: the text clear button should dissapear when there is no text in the entry. After you click on it, for example. See the search entries in sugar for reference.
But we can follow up that in another ticket, pushed as af31a44d . Thanks.
use the 'x' to clear the url entry