[nautilus/wip/antoniof/gtk4-pre-switch-regressions: 1/4] toolbar: Drop theatrics animation on the operations button
- From: António Fernandes <antoniof src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [nautilus/wip/antoniof/gtk4-pre-switch-regressions: 1/4] toolbar: Drop theatrics animation on the operations button
- Date: Wed, 15 Dec 2021 20:25:36 +0000 (UTC)
commit b463b1c2b46c60c51103a99ea75f5131d589bc37
Author: António Fernandes <antoniof gnome org>
Date: Wed Dec 15 14:58:34 2021 +0000
toolbar: Drop theatrics animation on the operations button
We have been using the copy-pasted code in src/animation to show an
animation around the operations button when it has just been revealed,
because CSS animations are clipped by GtkRevealer.
However, this copy-pasted code is burdersome to convert to GTK4, and
after the conversion we are going to have an easier time animating
widgets anyway.
So, in order to make the transition easier, take a temporary regression.
This effectively reverts commit 7f8cc0b8892105b24f944ac24248ba3b5f37ecbf,
but without reintroducing 5ce09ad04b94c4f1652afaa96852437ec40fc934
src/animation/egg-animation.c | 1183 --------------------------------------
src/animation/egg-animation.h | 74 ---
src/animation/egg-frame-source.c | 130 -----
src/animation/egg-frame-source.h | 29 -
src/animation/ide-box-theatric.c | 417 --------------
src/animation/ide-box-theatric.h | 30 -
src/animation/ide-cairo.c | 84 ---
src/animation/ide-cairo.h | 65 ---
src/meson.build | 8 -
src/nautilus-toolbar.c | 40 --
10 files changed, 2060 deletions(-)
---
diff --git a/src/meson.build b/src/meson.build
index 682d6f3a3..7ba3b44f6 100644
--- a/src/meson.build
+++ b/src/meson.build
@@ -53,14 +53,6 @@ libnautilus_sources = [
interface_prefix: 'org.gnome',
namespace: 'Nautilus'
),
- 'animation/egg-animation.c',
- 'animation/egg-animation.h',
- 'animation/egg-frame-source.c',
- 'animation/egg-frame-source.h',
- 'animation/ide-box-theatric.c',
- 'animation/ide-box-theatric.h',
- 'animation/ide-cairo.c',
- 'animation/ide-cairo.h',
'gtk/nautilusgtkplacesview.c',
'gtk/nautilusgtkplacesviewprivate.h',
'gtk/nautilusgtkplacesviewrow.c',
diff --git a/src/nautilus-toolbar.c b/src/nautilus-toolbar.c
index ca699e721..e1db6dbeb 100644
--- a/src/nautilus-toolbar.c
+++ b/src/nautilus-toolbar.c
@@ -25,9 +25,6 @@
#include <glib/gi18n.h>
#include <math.h>
-#include "animation/ide-box-theatric.h"
-#include "animation/egg-animation.h"
-
#include "nautilus-application.h"
#include "nautilus-bookmark.h"
#include "nautilus-file-operations.h"
@@ -46,9 +43,6 @@
#define NEEDS_ATTENTION_ANIMATION_TIMEOUT 2000 /*ms */
#define REMOVE_FINISHED_OPERATIONS_TIEMOUT 3 /*s */
-#define ANIMATION_X_GROW 30
-#define ANIMATION_Y_GROW 30
-
/* Just design, context at https://gitlab.gnome.org/GNOME/nautilus/issues/548#note_274131 */
typedef enum
@@ -547,40 +541,6 @@ update_operations (NautilusToolbar *self)
gtk_revealer_set_reveal_child (GTK_REVEALER (self->operations_revealer),
TRUE);
gtk_widget_queue_draw (self->operations_icon);
-
- /* Show the popover at start to increase visibility.
- * Check whether the toolbar is visible or not before showing the
- * popover. This can happens if the window has the disables-chrome
- * property set. */
- if (gtk_widget_is_visible (GTK_WIDGET (self)))
- {
- GtkAllocation rect;
- IdeBoxTheatric *theatric;
-
- gtk_widget_get_allocation (GTK_WIDGET (self->operations_button), &rect);
- theatric = g_object_new (IDE_TYPE_BOX_THEATRIC,
- "alpha", 0.9,
- "background", "#fdfdfd",
- "target", self->operations_button,
- "height", rect.height,
- "width", rect.width,
- "x", rect.x,
- "y", rect.y,
- NULL);
-
- egg_object_animate_full (theatric,
- EGG_ANIMATION_EASE_IN_CUBIC,
- 250,
- gtk_widget_get_frame_clock (GTK_WIDGET (self->operations_button)),
- g_object_unref,
- theatric,
- "x", rect.x - ANIMATION_X_GROW,
- "width", rect.width + (ANIMATION_X_GROW * 2),
- "y", rect.y - ANIMATION_Y_GROW,
- "height", rect.height + (ANIMATION_Y_GROW * 2),
- "alpha", 0.0,
- NULL);
- }
}
/* Since we removed the info widgets, we need to restore the focus */
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]