Code review comment for lp:~vanvugt/mir/set-subpixel

I'm a mild -1 on this. This is not a configuration property, and should not be a part of the configuration interface.

The shell has all the pieces it needs to implement overrides; that's an appropriate place for this functionality. I'd also fully support this being implemented in a Mir server configuration file - there is a broad need for some quirking infrastructure, and Mir could usefully provide that.

It's inappropriate for client configuration.