From d05c1f4677ab3dda3ad2b5964a554aec8611fdcc Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Tue, 21 May 2019 13:15:48 -0400 Subject: MetaFrameBorders: Add meta_frame_borders_clear Just a quick little commit to help clean things up for when we add invisible borders. Additionally, do a little housekeeping in preview-widget as well. https://bugzilla.gnome.org/show_bug.cgi?id=644930 NOTE: Patch copied from metacity and adapted for marco. upstream commit: https://gitlab.gnome.org/GNOME/metacity/commit/7d519b3f --- src/Makefile.am | 2 ++ src/core/common.c | 34 ++++++++++++++++++++++++++++++++++ src/include/common.h | 3 +++ src/ui/preview-widget.c | 22 +++++----------------- src/ui/preview-widget.h | 1 + src/ui/theme.c | 16 +++------------- src/ui/ui.c | 5 +---- 7 files changed, 49 insertions(+), 34 deletions(-) create mode 100644 src/core/common.c (limited to 'src') diff --git a/src/Makefile.am b/src/Makefile.am index 6d5c50f9..0b66147b 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -83,6 +83,7 @@ marco_SOURCES = \ core/workspace.h \ core/xprops.c \ include/xprops.h \ + core/common.c \ include/common.h \ include/core.h \ include/ui.h \ @@ -117,6 +118,7 @@ libmarco_private_la_SOURCES = \ ui/gradient.h \ core/util.c \ include/util.h \ + core/common.c \ include/common.h \ ui/preview-widget.c \ ui/preview-widget.h \ diff --git a/src/core/common.c b/src/core/common.c new file mode 100644 index 00000000..c7fe1012 --- /dev/null +++ b/src/core/common.c @@ -0,0 +1,34 @@ +/* -*- mode: C; c-file-style: "gnu"; indent-tabs-mode: nil; -*- */ + +/* Metacity X window decorations */ + +/* + * Copyright (C) 2001 Havoc Pennington + * Copyright (C) 2003, 2004 Red Hat, Inc. + * Copyright (C) 2005 Elijah Newren + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, see . + */ + +#include +#include "common.h" + +void +meta_frame_borders_clear (MetaFrameBorders *self) +{ + self->visible.top = 0; + self->visible.bottom = 0; + self->visible.left = 0; + self->visible.right = 0; +} diff --git a/src/include/common.h b/src/include/common.h index c49ad5de..18a4e12b 100644 --- a/src/include/common.h +++ b/src/include/common.h @@ -310,6 +310,9 @@ struct _MetaFrameBorders GtkBorder visible; }; +/* sets all dimensions to zero */ +void meta_frame_borders_clear (MetaFrameBorders *self); + /* should investigate changing these to whatever most apps use */ #define META_DEFAULT_ICON_SIZE 48 #define META_MIN_ICON_SIZE 6 diff --git a/src/ui/preview-widget.c b/src/ui/preview-widget.c index 6c819707..2928d3b1 100644 --- a/src/ui/preview-widget.c +++ b/src/ui/preview-widget.c @@ -98,10 +98,7 @@ meta_preview_init (MetaPreview *preview) META_FRAME_ALLOWS_SHADE | META_FRAME_ALLOWS_MOVE; - preview->borders.visible.left = -1; - preview->borders.visible.right = -1; - preview->borders.visible.top = -1; - preview->borders.visible.bottom = -1; + preview->borders_cached = FALSE; } GtkWidget* @@ -172,23 +169,17 @@ ensure_info (MetaPreview *preview) pango_font_description_free (font_desc); } - if (preview->borders.visible.top < 0) + if (!preview->borders_cached) { if (preview->theme) - { meta_theme_get_frame_borders (preview->theme, preview->type, preview->text_height, preview->flags, &preview->borders); - } else - { - preview->borders.visible.top = 0; - preview->borders.visible.bottom = 0; - preview->borders.visible.left = 0; - preview->borders.visible.right = 0; - } + meta_frame_borders_clear (&preview->borders); + preview->borders_cached = TRUE; } } @@ -362,10 +353,7 @@ clear_cache (MetaPreview *preview) preview->layout = NULL; } - preview->borders.visible.left = -1; - preview->borders.visible.right = -1; - preview->borders.visible.top = -1; - preview->borders.visible.bottom = -1; + preview->borders_cached = FALSE; } void diff --git a/src/ui/preview-widget.h b/src/ui/preview-widget.h index 11818588..898a226d 100644 --- a/src/ui/preview-widget.h +++ b/src/ui/preview-widget.h @@ -51,6 +51,7 @@ struct _MetaPreview int text_height; MetaFrameBorders borders; + guint borders_cached : 1; MetaButtonLayout button_layout; }; diff --git a/src/ui/theme.c b/src/ui/theme.c index 37298e42..299f1cc2 100644 --- a/src/ui/theme.c +++ b/src/ui/theme.c @@ -219,11 +219,6 @@ meta_frame_layout_new (void) layout->right_width = -1; layout->bottom_height = -1; - layout->invisible_border.left = 10; - layout->invisible_border.right = 10; - layout->invisible_border.bottom = 10; - layout->invisible_border.top = 10; - init_border (&layout->title_border); layout->title_vertical_pad = -1; @@ -430,10 +425,8 @@ meta_frame_layout_get_borders (const MetaFrameLayout *layout, if (flags & META_FRAME_FULLSCREEN) { - borders->visible.top = 0; - borders->visible.bottom = 0; - borders->visible.left = 0; - borders->visible.right = 0; + meta_frame_borders_clear (borders); + return; } } @@ -5656,10 +5649,7 @@ meta_theme_get_frame_borders (MetaTheme *theme, style = theme_get_style (theme, type, flags); - borders->visible.top = 0; - borders->visible.left = 0; - borders->visible.right = 0; - borders->visible.bottom = 0; + meta_frame_borders_clear (borders); /* Parser is not supposed to allow this currently */ if (style == NULL) diff --git a/src/ui/ui.c b/src/ui/ui.c index e7993625..cc26451e 100644 --- a/src/ui/ui.c +++ b/src/ui/ui.c @@ -685,10 +685,7 @@ meta_ui_theme_get_frame_borders (MetaUI *ui, } else { - borders->visible.top = 0; - borders->visible.bottom = 0; - borders->visible.left = 0; - borders->visible.right = 0; + meta_frame_borders_clear (borders); } if (style != NULL) -- cgit v1.2.1