PopupMenu section header size is untrustworthy

General JUCE discussion
1

Why, oh why, does HeaderItemComponent (for PopupMenu) call getIdealPopupMenuItemSize() and then just blatantly add two offsets to the result?

void getIdealSize (int& idealWidth, int& idealHeight) override
{
    getLookAndFeel().getIdealPopupMenuItemSize (getName(), false, -1, idealWidth, idealHeight);
    idealHeight += idealHeight / 2;
    idealWidth += idealWidth / 4;
}

I’d like to have full control over the size of my popup menu section headers. Can we please move this behaviour into a LookAndFeel call?

8 Likes
2

Seconded

3

+1

I still don’t see any way to change the height of a section header.

1 Like
4

+1
It’s still there

5

Thanks for the heads up.

You can find the relevant change on develop:

6

The comment on that commit does not appear to be correct, as it only has an effect on section headers?