Closed Bug 882568 Opened 11 years ago Closed 11 years ago

Remove unnecessary text about a bug not being part of last search

Categories

(Bugzilla :: User Interface, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: ashish, Assigned: glob)

Details

Attachments

(1 file)

If you arrive at a bug directly and not via search, the line "First Last Prev Next This bug is not in your last search results." is still shown. It has no links (understandably) and without that line, the interface would look cleaner on both the top and bottom of the page. Please look into removing that if possible. Thanks!
Attached patch patch v1Splinter Review
sounds reasonable.
Attachment #761892 - Flags: review?(dkl)
Assignee: ui → glob
Comment on attachment 761892 [details] [diff] [review] patch v1 Review of attachment 761892 [details] [diff] [review]: ----------------------------------------------------------------- Makes sense. r=dkl
Attachment #761892 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval4.4?
Shouldn't <div class="navigation"> be moved into [% IF my_search %]? I don't see the point to have an empty <div>.
Status: NEW → ASSIGNED
Flags: approval4.4? → approval4.4-
Target Milestone: --- → Bugzilla 5.0
(In reply to Frédéric Buclin from comment #3) > Shouldn't <div class="navigation"> be moved into [% IF my_search %]? I don't > see the point to have an empty <div>. Good point. Glob, please move the <div> inside the IF block on commit. LpSolit, can we still go ahead with approve+ then? dkl
a=LpSolit with <div> </div> moved inside [% IF %].
Flags: approval? → approval+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bugzilla/trunk/ modified template/en/default/bug/navigate.html.tmpl Committed revision 8641.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: