Beta 11 regressions with MenuBar and Button in VerticalLayout

Vaadin 6 / 7 / 8
1

Hi,

after upgrading to Vaadin7 Beta 11 i noticed some odd things.
The Text of the MenuItems didn’t show up any more.
After investigation i found, it was CSS specific.

This gives the error:

.v-menubar {
font-size: 125%;
}

This works:

.v-menubar {
font-size: large;
}

Another thing is a VerticalLayout with setWidth( 12, Unit.EM )
and added Buttons with setWidth( 100, Unit.PERCENTAGE )

The Layout is in the right place, but it is much to narrow.
So the Buttons are to narrow, too.

In addition, the Buttons have a different size depending on
enabled/disabled when i use the CSS

.v-button-wrap {
padding-left: 0;
padding-right: 3px;
}

I can try to add some short examples, if neccessary.

greetings
Ralf

2

Hi Ralf

Were you using Chameleon perhaps? If so, it may be related to this
post
 …

3

Hi,

Sorry, forgot to mention: i am using reindeer.

greetings
Ralf

4

This is probably related to a fix that was needed for IE - font size is set to zero on many parent elements and then set again correctly for the children. In your case, the 125% is probably calculated from zero.

In the future, using a font size variable and calculating the new size from that should help, but at the moment the required SASS arithmetic operations are not yet implemented (
#9354
). I have been looking into support for arithmetics, but other issues have had a higher priority now in late December.

This might be the same issue as above - the 12em is calculated from the wrong font size. This I would consider a bug, although perhaps not an easy one to fix correctly - please
create a ticket
 about this.

I would guess there might be a more specific rule overriding what you set in one of the cases - check with FireBug or other tools which rules are actually used in each case.

5

Hi,

New ticket
#10631
 for this.

greetings
Ralf