Closed Bug 390955 Opened 17 years ago Closed 10 years ago

Add some useful standard searches to the index page

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: mkanat, Assigned: LpSolit)

References

(Blocks 1 open bug)

Details

Attachments

(4 files, 1 obsolete file)

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.
Priority: -- → P2
Bug 130835 and its blockers already suggest this. No need to file a new bug.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
Target Milestone: Bugzilla 3.2 → ---
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.
Blocks: 130835
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Whiteboard: [blocker will fix if blocker is fixed first]
Target Milestone: --- → Bugzilla 4.0
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.
Target Milestone: Bugzilla 4.4 → Bugzilla 5.0
Assignee: mkanat → general
Attached patch patch, v1 (obsolete) — Splinter Review
Assignee: general → LpSolit
Status: REOPENED → ASSIGNED
Attachment #818802 - Flags: review?(dkl)
Attached image rss.png
Icon for feeds
Attached image 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.
dkl: ping?
Whiteboard: [blocker will fix if blocker is fixed first]
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/
Attachment #818802 - Flags: review?(dkl) → review-
(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?
Attached patch patch, v2Splinter Review
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).
Attachment #8356295 - Flags: review?(dkl)
Attachment #818802 - Attachment is obsolete: true
Comment on attachment 8356295 [details] [diff] [review]
patch, v2

r=dkl
Attachment #8356295 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval? → approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/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.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago10 years ago
Keywords: relnote
Resolution: --- → FIXED
I had to replace [ and ] in URLs by %5B and %5D respectively to make the HMTL5 validator happy:

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified template/en/default/index.html.tmpl
Committed revision 8857.
Blocks: 962416
Keywords: relnote
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: