Edgewall Software
Modify

Opened 16 years ago

Closed 16 years ago

#7272 closed defect (fixed)

Search bar/button appears over banner image

Reported by: Jason Winnebeck <jpwasp@…> Owned by: Jonas Borgström
Priority: high Milestone: 0.11
Component: general Version: 0.11-stable
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

After upgrading from r7086 of 0.11-stable to r7166, now the search bar appears on the left over the banner image, instead of right-justified. I tried reloading the page, thinking of possible cached CSS in my browser (Firefox 2), but still the search bar appears incorrectly.

Attachments (0)

Change History (4)

comment:1 by Jason Winnebeck <jpwasp@…>, 16 years ago

Based on debugging from the browser only (not looking at any Trac code), I found the following:

The problem appears to be this style code in trac.css:

/* Quick search */
#search {
 clear: both;
 font-size: 10px;
 height: 2.2em;
 margin: 0 0 1em;
 text-align: right;
}
#search input { font-size: 10px }
#search label { display: none }

However, the rendered output from Trac shows that the forms's ID is actually #quicksearch and not #search. I'm not sure which name was the original.

By inserting a custom CSS as the following:

#quicksearch {
 clear: both;
 font-size: 10px;
 height: 2.2em;
 margin: 0 0 1em;
 text-align: right;
}
#quicksearch input { font-size: 10px }
#quicksearch label { display: none }

My problem was solved. I did not try to update trac.css since I didn't want to touch the source for this problem, and my site was already using an additional custom CSS (I checked to make sure that was not the problem first).

comment:2 by Jason Winnebeck <jpwasp@…>, 16 years ago

It actually appears that the latest changeset, r7166, created this problem. Looking at the diffs, the "name" was quicksearch and replaced the old id of "search".

comment:3 by Jason Winnebeck <jpwasp@…>, 16 years ago

Looks like r7168 and r7172 fix this problem? I haven't tested it yet. ecarter and jhampton checked in changes, but I don't think they are aware of this ticket.

comment:4 by Eli Carter, 16 years ago

Resolution: fixed
Status: newclosed

Fixed in r7172 for 0.11-stable and r7173 for trunk.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström 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.