Switching from simple to advanced search loses the search text

I'm inclined to do a POST with the "Advanced search" link, rather than turn an href into a button, and have this just not work for the non-javascript heads. Launchpad already almost requires JS anyway, and loss of this functionality is only a small pain.

What do you think?

On Wed, Oct 11, 2006 at 04:07:35PM -0000, Brad Bollenbach wrote:
> I'm inclined to do a POST with the "Advanced search" link, rather than
> turn an href into a button, and have this just not work for the non-
> javascript heads. Launchpad already almost requires JS anyway, and loss
> of this functionality is only a small pain.
>
> What do you think?

How about a simpler solution to begin with; After you've searched for
"disk error", make the "Advanced search" link point to
"?advanced=1&field.searchtext=disk+error"?

I don't see why you want to POST the data? This bug is about switching to
advanced search after doing a simple search.

On 11-Oct-06, at 12:45 PM, Björn Tillenius wrote:

> On Wed, Oct 11, 2006 at 04:07:35PM -0000, Brad Bollenbach wrote:
>> I'm inclined to do a POST with the "Advanced search" link, rather
>> than
>> turn an href into a button, and have this just not work for the non-
>> javascript heads. Launchpad already almost requires JS anyway, and
>> loss
>> of this functionality is only a small pain.
>>
>> What do you think?
>
> How about a simpler solution to begin with; After you've searched for
> "disk error", make the "Advanced search" link point to
> "?advanced=1&field.searchtext=disk+error"?
>
> I don't see why you want to POST the data? This bug is about
> switching to
> advanced search after doing a simple search.

A user that enters some searchtext and then decides to switch to the
advanced search is more likely than not to want to keep the text they
entered. And POSTing from the link would make it easy to let the user
modify the current search filter, which could trivially be captured
in hidden fields.

On Wed, Oct 11, 2006 at 05:38:21PM -0000, Brad Bollenbach wrote:
> On 11-Oct-06, at 12:45 PM, Björn Tillenius wrote:
>
> > On Wed, Oct 11, 2006 at 04:07:35PM -0000, Brad Bollenbach wrote:
> >> I'm inclined to do a POST with the "Advanced search" link, rather
> >> than
> >> turn an href into a button, and have this just not work for the non-
> >> javascript heads. Launchpad already almost requires JS anyway, and
> >> loss
> >> of this functionality is only a small pain.
> >>
> >> What do you think?
> >
> > How about a simpler solution to begin with; After you've searched for
> > "disk error", make the "Advanced search" link point to
> > "?advanced=1&field.searchtext=disk+error"?
> >
> > I don't see why you want to POST the data? This bug is about
> > switching to
> > advanced search after doing a simple search.
>
> A user that enters some searchtext and then decides to switch to the
> advanced search is more likely than not to want to keep the text they
> entered. And POSTing from the link would make it easy to let the user
> modify the current search filter, which could trivially be captured
> in hidden fields.

Well, that's a valid use case, but I wonder how common it is (compared
to first doing a search)? I'd rather fix this bug using a simple fix
first, and then doing something fancy later.

And capturing the existing search filters doesn't requiring POSTing,
you can just as well keep that information in the "Advanced search"
link.

Brad and Bjorn, you're both right. :-) I should have been clearer in the initial report. I'll fix it.

It loses not just your search text, but also all of the other search criteria you may have established as a result of a previous or bookmarked advanced search or other means. For instance from

<https://bugs.edge.launchpad.net/bzr/+bugs?field.searchtext=binary&orderby=-date_last_updated&search=Search&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=FIXRELEASED&field.status%3Alist=INVALID&field.status%3Alist=WONTFIX&field.assignee=&field.bug_reporter=&field.omit_dupes=&field.has_patch=&field.has_no_package=>

when I click Advanced I'd expect to see all those status buttons already selected, but in fact the whole form is blank.