Project

General

Profile

Actions

Defect #4495

closed

Tab's 'border-bottom' not absent when selected (alternate)

Added by Felix Schäfer over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
UI
Target version:
Start date:
2009-12-28
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

In current trunk, the selected tab in project settings or redmine settings has a visible border-bottom, which wasn't there some revisions ago. This is for the alternate theme and current trunk, although I have narrowed down the change to somewhere between r3140 and r3211.


Files

defect4495.patch (1.75 KB) defect4495.patch patch for bottom-border problem Kevin Bosman, 2010-01-10 15:43
Actions #1

Updated by Mischa The Evil over 14 years ago

I can confirm the issue using source:/trunk@3255 together with the default theme. I was able to trace it back to r3174.

Actions #2

Updated by Eric Davis over 14 years ago

It's caused from this css:

#content .tabs {height: 2.6em; border-bottom: 1px solid #bbbbbb; margin-bottom:1.2em; position:relative; overflow:hidden;}

The tabs div has a bottom border that is visible above all of the tabs.

Actions #3

Updated by Kevin Bosman over 14 years ago

Mischa is correct: this behaviour was introduced by r3174.
The attached patch (defect4495.patch) should fix the problem in the default and alternate themes for Firefox and IE on Windows.
I haven't been able to test it on any other browsers (volunteers?)

Actions #4

Updated by Felix Schäfer over 14 years ago

Works on Safari/OS X (so probably every webkit browser) too, my css-foo isn't good enough to judge of the sanity of the patch though.

Actions #5

Updated by Felix Schäfer over 14 years ago

(forgot to mention that I tested this on alternate, not default, but that should work on it as well)

Actions #6

Updated by Jean-Philippe Lang over 14 years ago

  • Status changed from New to Closed
  • Target version set to 0.9.1
  • Resolution set to Fixed

I was aware of this issue but I didn't even try to fix it, thanks for the patch.
Tested on default theme with Firefox 3.5.7, IE 8 and committed in r3325.

Actions

Also available in: Atom PDF