Code review comment for lp:~brandontschaefer/unity/decor-dynamic-hi-dpi

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

76 + nux::Property<int> monitor;

Do we need a public property for this? It doesn't seem to be the case from what I see, so I'd just go for a Window::Impl private monitor_.

Other than that it seems good!

« Back to merge proposal