Edgewall Software
Modify

Opened 11 years ago

Closed 8 years ago

Last modified 8 years ago

#11292 closed enhancement (fixed)

search checkboxes should default to just tickets

Reported by: jtm.moon.forum.user+trac@… Owned by: Christian Boos
Priority: low Milestone: 1.3.1
Component: general Version: 0.12-stable
Severity: trivial Keywords: search
Cc: Branch:
Release Notes:

Remember last selected search filters in the session

API Changes:
Internal Changes:

Description

Problem

Most of the time a user only wants to search tickets. However, the Search page defaults to checking all available checkboxes (Tickets Wiki Changesets etc.). The user must manually uncheck these for every search. This is annoying.

Solutions

There are few ways to improve this, in order of simplicity

  1. tell users to remember the URL link i.e. https://trac/search?noquickjump=1&ticket=on
  2. Global hardcoded default to auto-check only the Tickets checkbox.
  3. Allow global Admininistration setting which allows picking which checkbox are auto-checked.
  4. Allow per-user setting to allow picking which checkbox are auto-checked.

This is a follow-on to this old email thread.

I usually search with the search button, but 99% of the time, I only want to search tickets so it would be nice to default the other checkboxes to unchecked.

I'd like to have this feature too. I think it could be quite useful to store the search option on a per-user basis, so that the default search options are restored from one search session to the other.

Attachments (1)

0001-search-save-last-selection-of-filters-in-the-session.patch (2.8 KB ) - added by Christian Boos 8 years ago.
store last selected search filters as comma-separated list in session, under key 'search.filters'

Download all attachments as: .zip

Change History (9)

in reply to:  description comment:1 by Ryan J Ollos, 11 years ago

Replying to jtm.moon.forum.user+trac@…:

Solutions

There are few ways to improve this, in order of simplicity

  1. tell users to remember the URL link i.e. https://trac/search?noquickjump=1&ticket=on
  2. Global hardcoded default to auto-check only the Tickets checkbox.
  3. Allow global Admininistration setting which allows picking which checkbox are auto-checked.
  4. Allow per-user setting to allow picking which checkbox are auto-checked.

The TracIni setting [search] default_disabled_filters allows disabling filters. See TracIni#search-section.

We could leave this ticket open for implementing a per-user preference. Another approach would be to have the state of the checkboxes persistent in the session data.

comment:2 by jtm.moon.forum.user+trac@…, 11 years ago

Awesome, thanks rjollos .

The TracIni setting [search] default_disabled_filters allows disabling filters

This is a solution for option 2. I think that is good enough.

I think you could close this ticket as worksforme (I cannot close it).

comment:3 by Ryan J Ollos, 11 years ago

Resolution: worksforme
Status: newclosed

Sounds good, I'll close as worksforme. I'm not convinced persistent session data would be a good approach. A user preference might be reasonable, but there are a number of other user preferences that should probably be implemented first, see comment:19:ticket:9807.

comment:4 by Christian Boos, 8 years ago

Some users really insist on having their one way of using search remembered…

Hence this quick patch, 0001-search-save-last-selection-of-filters-in-the-session.patch.

Ryan, as you expressed scepticism about this approach, I'm leaving the ticket closed. If you'd nevertheless be OK with the patch, let me know.

by Christian Boos, 8 years ago

store last selected search filters as comma-separated list in session, under key 'search.filters'

comment:5 by Ryan J Ollos, 8 years ago

I think it would be okay to apply the patch. I don't have a strong preference either way, and it's easy to change the state of the filters, even if there are many, after #12376.

comment:6 by Christian Boos, 8 years ago

Milestone: 1.3.1
Resolution: worksforme
Status: closedreopened

comment:7 by Christian Boos, 8 years ago

Release Notes: modified (diff)
Resolution: fixed
Status: reopenedclosed

I've added a functional test for the feature and committed it as r15214.

comment:8 by Christian Boos, 8 years ago

Owner: set to Christian Boos

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.