Underscores in some menu items not shown

[Peter Feichtinger]

Because all menu items for the dock item context menu are created with mnemonics enabled, underscores will not be shown.
This is a problem for window titles and for file names in the Trash docklet menu.
For folder items there's a workaround in the code (doubling all underscores).

The attached patch fixes the problem by introducing new factory methods for literal menu items (which should not use mnemonics).

Plank version 0.11.3 running on Arch Linux with xfwm4.

[Rico Tzschichholz (ricotz)]

Overall this seems like a proper solution (without breaking ABI).
Although pixbufs retrieved with DrawingService.load_icon() should not be treated with ar_scale() again.

[Peter Feichtinger (shippo)]

Since DrawingService.load_icon() presumably loads the pixbufs with the right size, they wouldn't be resized again.

[Rico Tzschichholz (ricotz)]

Either width *or* height are guarantied to match, but you can not rely on both. Although the checks in ar_scale() are sufficient to bail early in this case. Although it would be nice to avoid the calls of icon_size_lookup() and ar_scale() as it was before.

[Peter Feichtinger (shippo)]

I don't like the duplicated code, but you're right.
New patch attached.

[Rico Tzschichholz (ricotz)]

Thanks, pushed as r1587 with some minor additions.