(sorry I commented on the wrong line)

The logic for which connection is a display connection more complicated than this. Also, it may change in the future (e.g. if we had a ```<Display>``` tag for explicitly specifying the connection) and it would be great if this code did not rely on things implicitly. This also relies on the order of parsing a widget in LegacySkinParser calling setup before setupConnections (which we may change in the future).

In short, this code doesn't belong here. We shouldn't be calculating the display connection here just to use it in getDefaultDirectionOption. I think you should pass whether a display connection exists into getDefaultDirectionOption.