rlopez1j / xmonad

Automatically exported from code.google.com/p/xmonad
0 stars 0 forks source link

Layout.SubLayouts does not work with Layout.Combo #327

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. combineTwo (vertTiled) (styleTabbed) (Accordion)
   where
       vertTiled = ResizableTall 1 (2/100) (1/2) []
       styleTabbed = tabbed shrinkText myTabConfig
2. Use Layout.SubLayouts as recommended in the documentation (a modifier
for a ||| seperated list of layouts), and with modifiers from
WindowNavigation and BoringWindows

What is the expected output? What do you see instead?
The combineTwo layout is not displayed properly. It should work with all
layouts because combineTwo provides one layout, while subLayouts modifies
all layouts.

Original issue reported on code.google.com by OrbisVi...@gmail.com on 3 Nov 2009 at 12:37

GoogleCodeExporter commented 9 years ago
It's a known and difficult issue that not all layouts can be nested within 
others.

As described here, decorated layouts must not be used as the inner layout:
http://xmonad.org/xmonad-docs/xmonad-contrib/XMonad-Layout-SubLayouts.html#3

Instead, use
http://xmonad.org/xmonad-docs/xmonad-contrib/XMonad-Layout-SubLayouts.html#v%3As
ubTabbed
(or it's pieces) with a more pure layout, if you want decorations.

I believe that this can be type-system enforced by choosing a different type 
for the
inner layout to manage (ex. Int) which is just as Combo does (currently it is
Window), but I guess that isn't a substitute for a larger disclaimer (since the
errors would suggest to write the missing instance).

Original comment by vogt.a...@gmail.com on 3 Nov 2009 at 1:57

GoogleCodeExporter commented 9 years ago
I want to make sure these are the same issues.

In my case the tabbed layout *does* display properly. The accordion layout is
blank/empty//no_window.

This is using the subLayout modifier, even when not having created any 
subLayouts yet.

Original comment by OrbisVi...@gmail.com on 3 Nov 2009 at 5:06