Re: Shifty: Don't create tags on non-existing screens

2009-05-21 Thread Andreas Klöckner
On Mittwoch 20 Mai 2009, you wrote:
 On Thu, May 21, 2009 at 01:28, Andreas Klöckner li...@informa.tiker.net 
wrote:
 awesome ver?
3.3rc2

 shifty-master HEAD? (patch applies clean)
yes, HEAD

 The patch seems to be good but (before I push it) can you be a bit
 more specific as to what breaks (config snippets very helpful) if you
 run without it? I can only assume that the problem is that you're
 setting screen = 2 in somewhere in shifty.config and that breaks when
 running only on LVDS?

Yup, exactly.

Without the fix, all I can say is that windows with matches for the tags on 
the non-existent screen became inacessible--the apps are still running, but I 
could not figure out how to get the windows to show up. It was mildly 
frustrating. There was also some debug spew in .xsesion-errors, but I have no 
record of it right now.

 Btw, I've been using this sort of statement in rc.lua to force screen
 in various settings (shifty, naughty etc.) on my laptop, but I have to
 admit that I don't use multi-screen every day:

if screen.count() == 2 then LCD = 2 else LCD = 1 end

 Your solution might make more sense - shifty is supposed to be
 dynamic, so validation of screen could be done internally.

See it as guarding against user error--creating tags on non-existing screens 
is kind of pointless. On top of that, this fix makes dynamic dual-screen 
setups work with relative ease. What's not to like? :)

Andreas


signature.asc
Description: This is a digitally signed message part.


Re: Shifty: Don't create tags on non-existing screens

2009-05-21 Thread koniu
On Thu, May 21, 2009 at 15:45, Andreas Klöckner li...@informa.tiker.net wrote:
 See it as guarding against user error--creating tags on non-existing screens
 is kind of pointless. On top of that, this fix makes dynamic dual-screen
 setups work with relative ease. What's not to like? :)

Agreed and pushed - thanks :).

Also, please let me know if you experience any other issues/weirdness
with multi-screen setup and shifty - testing that properly is high on
my todo list and I could definitely use some hints from someone who
uses multiple screens on a more regular basis.

cheers,
koniu

--
To unsubscribe, send mail to awesome-devel-unsubscr...@naquadah.org.


Re: Shifty: Don't create tags on non-existing screens

2009-05-20 Thread koniu
On Thu, May 21, 2009 at 01:28, Andreas Klöckner li...@informa.tiker.net wrote:
 attached diff does what subject says--this makes shifty usable for me with a
 laptop with a varying number of screens attached.

Hi,

For the record please:
awesome ver?
shifty-master HEAD? (patch applies clean)

The patch seems to be good but (before I push it) can you be a bit
more specific as to what breaks (config snippets very helpful) if you
run without it? I can only assume that the problem is that you're
setting screen = 2 in somewhere in shifty.config and that breaks when
running only on LVDS?

Btw, I've been using this sort of statement in rc.lua to force screen
in various settings (shifty, naughty etc.) on my laptop, but I have to
admit that I don't use multi-screen every day:

   if screen.count() == 2 then LCD = 2 else LCD = 1 end

Your solution might make more sense - shifty is supposed to be
dynamic, so validation of screen could be done internally.

thanks,
koniu

--
To unsubscribe, send mail to awesome-devel-unsubscr...@naquadah.org.