Code review comment for lp:~evfool/midori/findbar

Hi Robert,

On 22/07/11 10:19, Robert Roth wrote:
> 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://bugs.launchpad.net/midori/+bug/772841
>
> For more details, see:
> https://code.launchpad.net/~evfool/midori/findbar/+merge/68801
>
> 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