Ticket #1577 (closed defect: fixed)

Opened 14 years ago

Last modified 14 years ago

Portal tabs display is wrong when the depth is greater than 1

Reported by: div Owned by: fguillaume
Priority: P1 Milestone: CPS 3.4.1
Component: CPSSkins Version: 3.4.0
Severity: normal Keywords:
Cc:

Description

If the portal tabs are longer than one line and wrap to the second line, the displaying is wrong (tested in Firefox 1.5).

Change History

comment:1 Changed 14 years ago by div

Actually, the problem is not related to the number of items, but to the depth used. By default, the depth=1 and there is no problem, but for any depth greater than 1, a space is shown to the left of the tab. The longer the depth, the longer the space.

comment:2 Changed 14 years ago by div

  • Summary changed from Portal tabs display is wrong when there are too many items to Portal tabs display is wrong when the depth is greater than 1

comment:3 Changed 14 years ago by jmorliaguet

can it be solved by adding a

margin-left: 0 !important

in the css corresponding to the horizontal tab display mode?

comment:4 Changed 14 years ago by ogrisel

This is a problem with the new "horizontal box without title" layout for the tabs slot that was recently added to be able to align the tabs to the right. I you switch back to "horizontal menu" layout, the artifact disapears (but you can no longuer align the tabs to the right).

The "horizontal menu" layout use a combination of "display: block" + "float: left" for the <ul/> tags whereas the "horizontal box without title" layout uses a "display: inline" style that is obviously more sensitive to the margin attribute.

comment:5 Changed 14 years ago by jmorliaguet

you could try adding:

.cpsskinsHBox ul li a {
  margin:0 ! important;
}

in cpsskins_common-css2.css.dtml

comment:6 Changed 14 years ago by div

Adding

margin-left: 0 !important;

fixes the problem in Firefox. Opera still has some issues, but unfortunately Opera doesn't count :)

comment:7 Changed 14 years ago by div

Fixed in [34724]

comment:8 Changed 14 years ago by div

  • Status changed from new to closed
  • Resolution set to fixed

comment:9 Changed 14 years ago by div

  • Component changed from CPS (global) to CPSSkins
Note: See TracTickets for help on using tickets.