Fixing Codacy nitpicks

OpenShot · May 20, 2024 · 2514bd5 · 2514bd5
1 parent 7d0fe24
commit 2514bd5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git 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
@@ -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
@@ -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;