Merge lp:~evfool/midori/findbar into lp:~vcs-imports/midori/master
Proposed by
Robert Roth
Status: | Merged |
---|---|
Merge reported by: | Cris Dywan |
Merged at revision: | not available |
Proposed branch: | lp:~evfool/midori/findbar |
Merge into: | lp:~vcs-imports/midori/master |
Diff against target: |
50 lines (+20/-4) 1 file modified
toolbars/midori-findbar.c (+20/-4) |
To merge this branch: | bzr merge lp:~evfool/midori/findbar |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
VCS imports | Pending | ||
Review via email: mp+68801@code.launchpad.net |
Description of the change
Improved findbar to only show the clear icon if there is something to clear (text longer than 1 char) (bug #772841), otherwise hide it, and also reset the primary icon to the find icon after clearing the entry, because if the stop icon was there and the entry was cleared it did not reset, thus we had an empty text entry with a red stop button.
To post a comment you must log in.
Hi Robert,
On 22/07/11 10:19, Robert Roth wrote: /bugs.launchpad .net/midori/ +bug/772841 /code.launchpad .net/~evfool/ midori/ findbar/ +merge/ 68801
> Robert Roth has proposed merging lp:~evfool/midori/findbar into lp:midori.
>
> Requested reviews:
> VCS imports (vcs-imports)
> Related bugs:
> Bug #772841 in Midori: "inline find - don't show the clear button if there is nothing to clear"
> https:/
>
> For more details, see:
> https:/
>
> Improved findbar to only show the clear icon if there is something to clear (text longer than 1 char) (bug #772841), otherwise hide it, and also reset the primary icon to the find icon after clearing the entry, because if the stop icon was there and the entry was cleared it did not reset, thus we had an empty text entry with a red stop button.
lp:midori is an import from the upstream version control system. They
don't use launchpad for development as far as I know. You probably want
to propose the merge in whatever upstream uses.
Cheers,
Jelmer