shdocvw: Remove useless menu separator

Frédéric Delanoy frederic.delanoy at gmail.com
Mon Aug 16 18:42:21 CDT 2010


On Mon, Aug 16, 2010 at 23:28, Octavian Voicu <octavian.voicu at gmail.com> wrote:
> Not sure if it's useless. I thing it's intended to separate the static
> part of the menu (Add to favorites) from the dynamically generated
> list of favorite items that follows (which isn't implemented yet).

In that case, shouldn't the menu separator also be auto-generated when
the list of (>=1) favorite items is generated?
How about when there's no favorite at all?
Separators should only be used to separate non-empty "sections" IMO;
otherwise they just look out of place.

Frédéric



More information about the wine-devel mailing list