390955 - Add some useful standard searches to the index page

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, enhancement, P2)

Description

[Max Kanat-Alexander]

The Songbird Bugzilla has some useful searches on its index page, even for users who aren't logged in. The ones that I think would be particularly useful in every Bugzilla are:

* Bugs changed in the last day.
* Bugs changed in the last week.
* Bugs resolved in the last week.

Also, these might be useful additions:

* Bugs resolved in the last day
* Bugs filed in the last day
* Bugs filed in the last week

They all have a little RSS icon next to them, which is also nice.

Comment 1

[Frédéric Buclin]

Bug 130835 and its blockers already suggest this. No need to file a new bug.

Comment 2

[Max Kanat-Alexander]

Yes, I know that, but this bug is much more focused and would be a lot easier to make and review.

If bug 130835 is resolved first, then we can mark this one as a dup again or mark it as fixed by blocker.

Comment 3

[Frédéric Buclin]

We are going to branch for Bugzilla 4.4 next week and this bug is too invasive or too risky to be accepted for 4.4 at this point. The target milestone is set to 5.0 which is our next major release.

I ask the assignee to reassign the bug to the default assignee if you don't plan to work on this bug in the near future, to make it clearer which bugs should be fixed by someone else on time for 5.0.

Comment 4

[Frédéric Buclin]

Attachment: patch, v1

Comment 5

[Frédéric Buclin]

Attachment: rss.png

Icon for feeds

Comment 6

[Frédéric Buclin]

Attachment: screenshot

- Instead of a "Bugzilla User's guide" link below the QuickSearch box, I added a 4th icon named "Documentation";
- I removed the link to the QuickSearch plugin. Browsers already provide other ways to add it (e.g. by clicking the search box of the browser);
- I removed the link to release notes. IMO, that's not the right place for it (and this is only useful to admins. The average user doesn't care);
- For queries specific to the logged in user, the number of bugs is displayed. If the user is logged out, the first 3 queries are not displayed.

Comment 7

[Frédéric Buclin]

dkl: ping?

Comment 8

[David Lawrence [:dkl]]

Comment on attachment 818802 [details] [diff] [review]
patch, v1

Review of attachment 818802 [details] [diff] [review]:
-----------------------------------------------------------------

> - Instead of a "Bugzilla User's guide" link below the QuickSearch box, I
> added a 4th icon named "Documentation";

I would like to see this as optional, as not all installations host the full
documentation such as BMO. Could we check for the existence of using.html in index.cgi
and then set a flag for the template to use?

::: skins/standard/index.css
@@ +71,5 @@
> +    #enter_bug { background: url(index/file-a-bug.png)  no-repeat; }
> +    #query     { background: url(index/search.png)      no-repeat; }
> +    #account   { background: url(index/new-account.png) no-repeat; }
> +    #help      { background: url(index/help.png)        no-repeat; }
> +

Add:

    #common_queries p { 
        font-weight: bold;
    }
    #common_queries ul {
        list-style: none;
        padding-left: 1.5em;
    }
    #common_queries li a { 
        text-decoration: none; 
    }

::: template/en/default/index.html.tmpl
@@ +141,5 @@
> +      [% Hook.process('additional_links') %]
> +    </ul>
> +  </div>
> +
> +  <div>

<div id="common_queries">

@@ +142,5 @@
> +    </ul>
> +  </div>
> +
> +  <div>
> +    <p>Common queries:</p>

s/queries:/Queries/

Comment 9

[Frédéric Buclin]

(In reply to David Lawrence [:dkl] from comment #8)
> I would like to see this as optional, as not all installations host the full
> documentation such as BMO. Could we check for the existence of using.html in
> index.cgi

This icon is enclosed in [% IF Param("docs_urlbase") %] which does exactly that. All links pointing to the documentation in the header and footer are controlled by this parameter.


>     #common_queries p { 
>         font-weight: bold;
>     }

Why do you want it bold?

Comment 10

[Frédéric Buclin]

Attachment: patch, v2

If we want Common Queries bold, then it's a title, so I use <h4> instead which is cleaner (and more usable for screen readers).

Comment 11

[David Lawrence [:dkl]]

Comment on attachment 8356295 [details] [diff] [review]
patch, v2

r=dkl

Comment 12

[Frédéric Buclin]

Committing to: bzr+ssh://[e-mail]/bugzilla/trunk/
modified index.cgi
added images/rss.png
modified skins/standard/global.css
modified skins/standard/index.css
modified template/en/default/index.html.tmpl
Committed revision 8851.

Comment 13

[Frédéric Buclin]

Attachment: replace [ and ] by %5B and %5D

I had to replace [ and ] in URLs by %5B and %5D respectively to make the HMTL5 validator happy:

Committing to: bzr+ssh://[e-mail]/bugzilla/trunk/
modified template/en/default/index.html.tmpl
Committed revision 8857.