From 2514bd579f2e1155c4d8045c2b1c1331c6bc9c1e Mon Sep 17 00:00:00 2001 From: Jonathan Thomas Date: Mon, 20 May 2024 10:54:08 -0500 Subject: [PATCH] Fixing Codacy nitpicks --- src/themes/base.py | 4 ++-- src/timeline/js/directives/clip.js | 2 +- src/timeline/js/directives/transition.js | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/themes/base.py b/src/themes/base.py index fc2b8e460..853ff250d 100644 --- a/src/themes/base.py +++ b/src/themes/base.py @@ -75,7 +75,7 @@ def set_dock_margins(self, content_margins=None, layout_margins=None, object_nam # Set content margins on the QDock Layout (which has additional margins) child.layout().setContentsMargins(*layout_margins) - def set_toolbar_buttons(self, toolbar, icon_size=24, settings=[]): + def set_toolbar_buttons(self, toolbar, icon_size=24, settings=None): """Iterate through toolbar button settings, and apply them to each button. [{"text": "", "icon": ""},...] """ @@ -138,7 +138,7 @@ def apply_theme(self): # Hide main window status bar self.app.window.statusBar.hide() - from classes import info, ui_util + from classes import info from classes.logger import log from PyQt5.QtGui import QFont, QFontDatabase diff --git a/src/timeline/js/directives/clip.js b/src/timeline/js/directives/clip.js index 049480535..6448d4b25 100644 --- a/src/timeline/js/directives/clip.js +++ b/src/timeline/js/directives/clip.js @@ -346,7 +346,7 @@ App.directive("tlClip", function ($timeout) { var y_offset = ui.position.top - previous_y; // Move the bounding box and apply snapping rules - var results = moveBoundingBox(scope, previous_x, previous_y, x_offset, y_offset, ui.position.left, ui.position.top, item_type="clip",cursor_offset=initialOffset); + var results = moveBoundingBox(scope, previous_x, previous_y, x_offset, y_offset, ui.position.left, ui.position.top, "clip", initialOffset); x_offset = results.x_offset; y_offset = results.y_offset; diff --git a/src/timeline/js/directives/transition.js b/src/timeline/js/directives/transition.js index 9b23de151..3d65119ca 100644 --- a/src/timeline/js/directives/transition.js +++ b/src/timeline/js/directives/transition.js @@ -290,7 +290,7 @@ App.directive("tlTransition", function () { var y_offset = ui.position.top - previous_y; // Move the bounding box and apply snapping rules - var results = moveBoundingBox(scope, previous_x, previous_y, x_offset, y_offset, ui.position.left, ui.position.top, item_type="clip",cursor_offset=initialOffset); + var results = moveBoundingBox(scope, previous_x, previous_y, x_offset, y_offset, ui.position.left, ui.position.top, "clip", initialOffset); x_offset = results.x_offset; y_offset = results.y_offset;