Re: weston never calls drmModeCrtcSetGamma() by default

2013-12-01 Thread Michel Dänzer
On Son, 2013-12-01 at 21:42 -0500, Jasper St. Pierre wrote: > Well yes, if you don't load the color management module, you won't get > color management. Seems to be what I'd expect. No colour management is fine, but a 16bpp gamma ramp (let alone an 8bpp palette) just doesn't look very good in 32bp

Re: weston never calls drmModeCrtcSetGamma() by default

2013-12-01 Thread Jasper St. Pierre
Well yes, if you don't load the color management module, you won't get color management. Seems to be what I'd expect. On Sun, Dec 1, 2013 at 9:30 PM, Michel Dänzer wrote: > > Without loading the cms-colord module, weston never calls > drmModeCrtcSetGamma(). This means that weston's gamma ramp i

weston never calls drmModeCrtcSetGamma() by default

2013-12-01 Thread Michel Dänzer
Without loading the cms-colord module, weston never calls drmModeCrtcSetGamma(). This means that weston's gamma ramp is whatever was set previously. This is particularly bad when weston is launched from a console using a different depth than weston itself. -- Earthling Michel Dänzer

Re: [PATCH 01/16] xdg_shell: Adding a new shell protocol.

2013-12-01 Thread Bill Spitzak
On 11/29/2013 04:09 PM, Kristian Høgsberg wrote: it sounds like there's something we're not capturing here: dialogs that are transient_for will be attached by mutter, but floating toolboxes are also transient_for but doesn't get attached. I'm guessing mutter also looks at wether the window in qu

Re: [PATCH 01/16] xdg_shell: Adding a new shell protocol.

2013-12-01 Thread Bill Spitzak
On 11/29/2013 02:43 PM, Kristian Høgsberg wrote: - set_transient_for is not a great name. I know it comes from ICCCM, but I wonder if we can find a better name. If it's just stacking, then maybe just set_aboe? How about "parent"? Though I guess "above" is acceptable... Certainly t