Edgewall Software
Modify

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#9269 closed enhancement (fixed)

[PATCH] Jumping site-header

Reported by: Mikael Relbe Owned by: Mikael Relbe
Priority: highest Milestone: 0.12
Component: rendering Version: 0.12dev
Severity: trivial Keywords: css
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Problem appears when not using a logo for a site, but just a header: The site header jumps around when switching between Browse Source and other functions (Wiki, Tickets, etc). The problem is that "trunk/trac/htdocs/css/browser.css" defines padding for "h1", which is inherited by "#header h1" since it lacks padding definition.

(Yes Christian, this is the same that I mailed you directly, I was thinking it would be easier to get fixed via a ticket after all.)

Attachments (1)

minor-css-correction_trunk-9478.patch (647 bytes ) - added by Mikael Relbe 14 years ago.
This patch fixes the problem

Download all attachments as: .zip

Change History (8)

by Mikael Relbe, 14 years ago

This patch fixes the problem

comment:1 by Mikael Relbe, 14 years ago

Summary: Jumping site-header[Patch] Jumping site-header

comment:2 by Remy Blank, 14 years ago

Milestone: 0.120.12.1

Only critical tickets for 0.12.

comment:3 by Mikael Relbe, 14 years ago

Owner: set to Christian Boos
Priority: normalhighest
Severity: normaltrivial
Summary: [Patch] Jumping site-header[PATCH] Jumping site-header

This tiny patch is ready for being merged with trunk.

comment:4 by Christian Boos, 14 years ago

Please just show me how to reproduce this (no logo for a site, but just a header) easily ;-)

comment:5 by Mikael Relbe, 14 years ago

Create a new site having no logo but showing a project name in trac.ini:

[header_logo]
;alt = (please configure the [header_logo] section in trac.ini)
;height = -1
;link = 
src = 
;width = -1

[project]
name = Example Project

"Example Project" should now be shown instead of the non-existing icon, which jumps around when you switch between Browse Source and other resources.

comment:6 by Christian Boos, 14 years ago

Milestone: 0.12.10.12
Resolution: fixed
Status: newclosed

At long last… problem reproduced, fix tested and patch applied in [9654]. Thanks!

I actually never realized you have the project's name shown as a heading when you don't have a logo configured ;-)

comment:7 by Christian Boos, 14 years ago

Owner: changed from Christian Boos to Mikael Relbe

Modify Ticket

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