From a98cc194add3bfd023be3e18f3904fc7a6219c70 Mon Sep 17 00:00:00 2001 From: Tomas Janousek Date: Fri, 1 Jan 2021 23:12:54 +0100 Subject: [PATCH] X.L.NoBorders: Fix docs of data Ambiguity In c6cdb77e3b3d6a3842b914a0cf64357f03fcf0c1, several new constructors were added but not "In order of increasing ambiguity (less borders more frequently)" as the docs for data Ambiguity say. Correct the order and clarify that Never only considers ambiguity of tiled windows (as in the doc for smartBorders). Fixes: c6cdb77e3b3d ("'XMonad.Layout.NoBorders': various improvements:") Related: https://github.com/xmonad/xmonad-contrib/issues/280 --- XMonad/Layout/NoBorders.hs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/XMonad/Layout/NoBorders.hs b/XMonad/Layout/NoBorders.hs index 18f0a6d4..2f056699 100644 --- a/XMonad/Layout/NoBorders.hs +++ b/XMonad/Layout/NoBorders.hs @@ -301,9 +301,6 @@ data Ambiguity -- ^ This constructor is used to combine the borderless windows -- provided by the SetsAmbiguous instances from two other 'Ambiguity' -- data types. - | OnlyScreenFloat - -- ^ Only remove borders on floating windows that cover the whole - -- screen. | OnlyLayoutFloatBelow -- ^ Like 'OnlyLayoutFloat', but only removes borders if no window -- stacked below remains visible. Considers all floating windows on the @@ -314,9 +311,12 @@ data Ambiguity | OnlyLayoutFloat -- ^ Only remove borders on floating windows that exactly cover the -- parent layout rectangle. + | OnlyScreenFloat + -- ^ Only remove borders on floating windows that cover the whole + -- screen. | Never - -- ^ Never remove borders when ambiguous: this is the same as - -- smartBorders. + -- ^ Like 'OnlyScreenFloat', and also remove borders of tiled windows + -- when not ambiguous: this is the same as 'smartBorders'. | EmptyScreen -- ^ Focus in an empty screen does not count as ambiguous. | OtherIndicated