wm_shell does not use display size.

abyss7 / chromium-wm

Automatically exported from code.google.com/p/chromium-wm

1 stars 0 forks source link

wm_shell does not use display size. #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

If there is no window manager present I want the wm_shell to open at the 
resolution provided by my display.

With the attached patch, if you test in your current WM, nothing changes. If 
you test on another X server, you'll get the wm_shell at the correct size.

There are a few issues with running as the base window manager, there is a lot 
of screen tearing when you drag the "Hello X Server" window around. Along with 
that, you can only drag the other window part way across the screen (3/4? 
maybe) before it hits a will and thinks it's at the right side.

Original issue reported on code.google.com by dsinclair@chromium.org on 3 Mar 2013 at 4:49

Attachments:

bounds.patch

GoogleCodeExporter commented 9 years ago

Cool, can you throw up the patch on codereview.appspot.com? You can use the 
upload.py script here: https://codereview.appspot.com/static/upload.py to add 
it to the tracker. Then please add reveman@ and myself as reviewers. I will add 
something to the wiki for this!

I think maybe BUG=chromium-wm:2 would point it at this, but let's find out.

Original comment by danakj@chromium.org on 3 Mar 2013 at 7:05

GoogleCodeExporter commented 9 years ago

To be clear however, I think that wm_shell is meant only to be a test 
environment, so I'm not sure how useful it is to run it full screen. To get 
chromium WM running, the intention is you'd run chrome, similar to how we run 
it for chrome os. We need some instructions for how to do that, if it is 
possible yet, in the wiki too.

Original comment by danakj@chromium.org on 3 Mar 2013 at 7:06

GoogleCodeExporter commented 9 years ago

I'll make 2 new bugs for documentation. One for code review, and one for 
running chromium as a WM.

Original comment by danakj@chromium.org on 3 Mar 2013 at 7:07

GoogleCodeExporter commented 9 years ago

Done: https://code.google.com/p/chromium-wm/issues/detail?id=3 and 
https://code.google.com/p/chromium-wm/issues/detail?id=4

Original comment by danakj@chromium.org on 3 Mar 2013 at 7:08

GoogleCodeExporter commented 9 years ago

Original comment by danakj@chromium.org on 4 Mar 2013 at 12:01

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

Original comment by danakj@chromium.org on 4 Mar 2013 at 12:05

Changed state: Assigned

GoogleCodeExporter commented 9 years ago

wm_shell is just a test shell and shouldn't care about display size.

Original comment by reveman@chromium.org on 4 Mar 2013 at 1:30

Changed state: WontFix