Remove unused constants

daemon-gtk3/BackgroundMenu.vala

@@ -4,8 +4,8 @@
  */
 
 public class Gala.Daemon.BackgroundMenu : Gtk.Menu {
-    public const string ACTION_GROUP_PREFIX = "background-menu";
-    public const string ACTION_PREFIX = ACTION_GROUP_PREFIX + ".";
+    private const string ACTION_GROUP_PREFIX = "background-menu";
+    private const string ACTION_PREFIX = ACTION_GROUP_PREFIX + ".";
 
     construct {
         var change_wallpaper = new Gtk.MenuItem.with_label (_("Change Wallpaper…")) {

daemon/MonitorLabel.vala

@@ -4,7 +4,6 @@
  */
 
  public class Gala.Daemon.MonitorLabel : Gtk.Window {
-    private const int SPACING = 12;
     private const string COLORED_STYLE_CSS = """
     .%s {
         background-color: alpha(%s, 0.8);

plugins/pip/PopupWindow.vala

@@ -10,7 +10,6 @@ public class Gala.Plugins.PIP.PopupWindow : Clutter.Actor {
     private const uint FADE_OUT_TIMEOUT = 200;
     private const float MINIMUM_SCALE = 0.1f;
     private const float MAXIMUM_SCALE = 1.0f;
-    private const int SCREEN_MARGIN = 0;
     private const float OFF_SCREEN_PERCENT = 0.5f;
     private const int OFF_SCREEN_VISIBLE_PIXELS = 80;
 
@@ -90,11 +89,11 @@ public class Gala.Plugins.PIP.PopupWindow : Clutter.Actor {
 
         float x_position, y_position;
         if (Clutter.get_default_text_direction () == Clutter.TextDirection.RTL) {
-            x_position = SCREEN_MARGIN + workarea_rect.x;
+            x_position = workarea_rect.x;
         } else {
-            x_position = workarea_rect.x + workarea_rect.width - SCREEN_MARGIN - width;
+            x_position = workarea_rect.x + workarea_rect.width - width;
         }
-        y_position = workarea_rect.y + workarea_rect.height - SCREEN_MARGIN - height;
+        y_position = workarea_rect.y + workarea_rect.height - height;
 
         set_position (x_position, y_position);
 
@@ -425,10 +424,10 @@ public class Gala.Plugins.PIP.PopupWindow : Clutter.Actor {
 
         var workarea_rect = display.get_workspace_manager ().get_active_workspace ().get_work_area_all_monitors ();
 
-        var screen_limit_start_x = workarea_rect.x + SCREEN_MARGIN;
-        var screen_limit_end_x = workarea_rect.x + workarea_rect.width - SCREEN_MARGIN - width;
-        var screen_limit_start_y = workarea_rect.y + SCREEN_MARGIN;
-        var screen_limit_end_y = workarea_rect.y + workarea_rect.height - SCREEN_MARGIN - height;
+        var screen_limit_start_x = workarea_rect.x;
+        var screen_limit_end_x = workarea_rect.x + workarea_rect.width - width;
+        var screen_limit_start_y = workarea_rect.y;
+        var screen_limit_end_y = workarea_rect.y + workarea_rect.height - height;
 
         var duration = Utils.get_animation_duration (300);
 

src/NotificationStack.vala

@@ -18,7 +18,6 @@
 
 public class Gala.NotificationStack : Object {
     private const string TRANSITION_ENTRY_NAME = "entry";
-    private const string TRANSITION_MOVE_STACK_ID = "move-stack";
 
     // we need to keep a small offset to the top, because we clip the container to
     // its allocations and the close button would be off for the first notification

src/Widgets/MultitaskingView/WorkspaceClone.vala

@@ -95,7 +95,7 @@ public class Gala.WorkspaceClone : ActorTarget {
     /**
      * The offset of the scaled background to the bottom of the monitor bounds
      */
-    public const int BOTTOM_OFFSET = 100;
+    private const int BOTTOM_OFFSET = 100;
 
     /**
      * The offset of the scaled background to the top of the monitor bounds

src/Widgets/SessionLocker.vala

@@ -56,7 +56,7 @@ public class Gala.SessionLocker : Clutter.Actor {
     // is about to turn off
     public const uint LONG_ANIMATION_TIME = 3000;
     // Animation length used for manual lock action (i.e. Super+L or GUI action)
-    public const uint SHORT_ANIMATION_TIME = 300;
+    private const uint SHORT_ANIMATION_TIME = 300;
 
     private const string LOCK_ENABLED_KEY = "lock-enabled";
     private const string LOCK_PROHIBITED_KEY = "disable-lock-screen";

src/Zoom.vala

@@ -9,7 +9,6 @@ public class Gala.Zoom : Object, GestureTarget, RootTarget {
     private const float MIN_ZOOM = 1.0f;
     private const float MAX_ZOOM = 10.0f;
     private const float SHORTCUT_DELTA = 0.5f;
-    private const int ANIMATION_DURATION = 300;
     private const uint MOUSE_POLL_TIME = 50;
 
     public WindowManager wm { get; construct; }