Valo 7.3.3 - incorrect height of column headers in TreeTable

Vaadin 6 / 7 / 8
1

In Valo tree table v-table-caption-container has box-sizing: content-box, so min-height: $v-unit-size in 7.3.3 leads to BIG ugly column header.

import com.vaadin.annotations.Theme;
import com.vaadin.server.VaadinRequest;
import com.vaadin.ui.TreeTable;
import com.vaadin.ui.UI;
import com.vaadin.ui.VerticalLayout;
import com.vaadin.ui.themes.ValoTheme;

@Theme(ValoTheme.THEME_NAME)
public class TestUI extends UI {

    @Override
    protected void init(VaadinRequest request) {
        VerticalLayout layout = new VerticalLayout();
        layout.setSizeFull();

        TreeTable treeTable = new TreeTable();
        treeTable.setSizeFull();

        treeTable.addContainerProperty("Name", String.class, "");

        treeTable.addItem(new Object[]{"Root"}, 0);
        treeTable.addItem(new Object[]{"Branch 1"}, 1);
        treeTable.addItem(new Object[]{"Branch 2"}, 2);
        treeTable.addItem(new Object[]{"Leaf 1"}, 3);
        treeTable.addItem(new Object[]{"Leaf 2"}, 4);
        treeTable.addItem(new Object[]{"Leaf 3"}, 5);
        treeTable.addItem(new Object[]{"Leaf 4"}, 6);

        treeTable.setParent(1, 0);
        treeTable.setParent(2, 0);
        treeTable.setParent(3, 1);
        treeTable.setParent(4, 1);
        treeTable.setParent(5, 2);
        treeTable.setParent(6, 2);

        layout.addComponent(treeTable);
        setContent(layout);
    }
}
2

3

Thanks for pointing that out. Apparently we’re missing a test for the Valo style TreeTable. I’ll create a ticket for this.

4


http://dev.vaadin.com/ticket/15121