patch: give WindowNavigation an empty description
Hi all, The WindowNavigation layout modifier was using the default "show" as its description, leading to a lengthy mess when displaying the current layout (using e.g. DynamicLog). This patch changes the description to be empty, the reasoning being that I know whether I'm using WindowNavigation or not and don't need to see it in front of the layout. I've also included a small change to LayoutModifier so that an extra space is not added before the layout description when the modifier description is empty. -Brent
On Thu, Oct 18, 2007 at 02:37:35PM -0400, Brent Yorgey wrote:
Hi all,
The WindowNavigation layout modifier was using the default "show" as its description, leading to a lengthy mess when displaying the current layout (using e.g. DynamicLog). This patch changes the description to be empty, the reasoning being that I know whether I'm using WindowNavigation or not and don't need to see it in front of the layout. I've also included a small change to LayoutModifier so that an extra space is not added before the layout description when the modifier description is empty.
Thanks for the contribution! I've pushed a somewhat modified patch, which changes the default for layout modifiers to be an empty description. I don't see where this could hurt, and doubt that the show instance was ever what was wanted. -- David Roundy Department of Physics Oregon State University
David Roundy wrote:
On Thu, Oct 18, 2007 at 02:37:35PM -0400, Brent Yorgey wrote:
Hi all,
The WindowNavigation layout modifier was using the default "show" as its description, leading to a lengthy mess when displaying the current layout (using e.g. DynamicLog). This patch changes the description to be empty, the reasoning being that I know whether I'm using WindowNavigation or not and don't need to see it in front of the layout. I've also included a small change to LayoutModifier so that an extra space is not added before the layout description when the modifier description is empty.
Thanks for the contribution! I've pushed a somewhat modified patch, which changes the default for layout modifiers to be an empty description. I don't see where this could hurt, and doubt that the show instance was ever what was wanted.
Great! That was one of the things on my wishlist. Thanks for that (to both of you) :-) -- Christian Thiemann mail@christian-thiemann.de
participants (3)
-
Brent Yorgey -
Christian Thiemann -
David Roundy