setiastrosuitepro 1.6.0__py3-none-any.whl → 1.6.4.post1__py3-none-any.whl

setiastro/saspro/ops/settings.py

@@ -5,6 +5,9 @@ from PyQt6.QtWidgets import (
 from PyQt6.QtCore import QSettings, Qt
 import pytz  # for timezone list
 
+# i18n support
+from setiastro.saspro.i18n import get_available_languages, get_saved_language, save_language
+
 
 class SettingsDialog(QDialog):
     """
@@ -13,8 +16,11 @@ class SettingsDialog(QDialog):
     """
     def __init__(self, parent, settings: QSettings):
         super().__init__(parent)
-        self.setWindowTitle("Preferences")
+        self.setWindowTitle(self.tr("Preferences"))
         self.settings = settings
+        
+        # Ensure we don't delete on close, so we can cache it
+        self.setAttribute(Qt.WidgetAttribute.WA_DeleteOnClose, False)
 
         # ---- Existing fields (paths, checkboxes, etc.) ----
         self.le_graxpert = QLineEdit()
@@ -22,23 +28,13 @@ class SettingsDialog(QDialog):
         self.le_starnet  = QLineEdit()
         self.le_astap    = QLineEdit()
 
-        self.chk_updates_startup = QCheckBox("Check for updates on startup")
-        self.chk_updates_startup.setChecked(
-            self.settings.value("updates/check_on_startup", True, type=bool)
-        )
+        self.chk_updates_startup = QCheckBox(self.tr("Check for updates on startup"))
 
         self.le_updates_url = QLineEdit()
         self.le_updates_url.setPlaceholderText("Raw JSON URL (advanced)")
-        self.le_updates_url.setText(
-            self.settings.value(
-                "updates/url",
-                "https://raw.githubusercontent.com/setiastro/setiastrosuitepro/main/updates.json",
-                type=str
-            )
-        )
 
-        btn_reset_updates_url = QPushButton("Reset")
-        btn_reset_updates_url.setToolTip("Restore default updates URL")
+        btn_reset_updates_url = QPushButton(self.tr("Reset"))
+        btn_reset_updates_url.setToolTip(self.tr("Restore default updates URL"))
         btn_reset_updates_url.clicked.connect(
             lambda: self.le_updates_url.setText(
                 "https://raw.githubusercontent.com/setiastro/setiastrosuitepro/main/updates.json"
@@ -46,8 +42,8 @@ class SettingsDialog(QDialog):
         )
 
         # Optional: “Check Now…” button
-        self.btn_check_now = QPushButton("Check Now…")
-        self.btn_check_now.setToolTip("Run an update check immediately")
+        self.btn_check_now = QPushButton(self.tr("Check Now…"))
+        self.btn_check_now.setToolTip(self.tr("Run an update check immediately"))
         self.btn_check_now.setVisible(hasattr(parent, "_check_for_updates_async"))
         self.btn_check_now.clicked.connect(self._check_updates_now_clicked)
 
@@ -57,37 +53,31 @@ class SettingsDialog(QDialog):
         row_updates_url.addWidget(btn_reset_updates_url)
         row_updates_url.addWidget(self.btn_check_now)
 
-        self.chk_save_shortcuts = QCheckBox("Save desktop shortcuts on exit")
-        self.chk_save_shortcuts.setChecked(
-            self.settings.value("shortcuts/save_on_exit", True, type=bool)
-        )
+        self.chk_save_shortcuts = QCheckBox(self.tr("Save desktop shortcuts on exit"))
 
         self.cb_theme = QComboBox()
         # Order: Dark, Gray, Light, System, Custom
         self.cb_theme.addItems(["Dark", "Gray", "Light", "System", "Custom"])
 
-        theme_val = (self.settings.value("ui/theme", "system", type=str) or "system").lower()
-        index_map = {"dark": 0, "gray": 1, "light": 2, "system": 3, "custom": 4}
-        self.cb_theme.setCurrentIndex(index_map.get(theme_val, 2))
-
         # "Customize…" button for custom theme
-        self.btn_theme_custom = QPushButton("Customize…")
-        self.btn_theme_custom.setToolTip("Edit custom colors and font")
-        self.btn_theme_custom.setEnabled(theme_val == "custom")
+        self.btn_theme_custom = QPushButton(self.tr("Customize…"))
+        self.btn_theme_custom.setToolTip(self.tr("Edit custom colors and font"))
         self.btn_theme_custom.clicked.connect(self._open_theme_editor)
 
         # Keep button enabled state in sync with combo
         self.cb_theme.currentIndexChanged.connect(self._on_theme_changed)
 
-        self.le_graxpert.setText(self.settings.value("paths/graxpert", "", type=str))
-        self.le_cosmic.setText(self.settings.value("paths/cosmic_clarity", "", type=str))
-        self.le_starnet.setText(self.settings.value("paths/starnet", "", type=str))
-        self.le_astap.setText(self.settings.value("paths/astap", "", type=str))
+        # ---- Language selector ----
+        self.cb_language = QComboBox()
+        self._lang_codes = list(get_available_languages().keys())  # ["en", "it", "fr", "es"]
+        self._lang_names = list(get_available_languages().values())  # ["English", "Italiano", ...]
+        self.cb_language.addItems(self._lang_names)
+        self._initial_language = "en" # placeholder, set in refresh_ui
 
-        btn_grax  = QPushButton("Browse…"); btn_grax.clicked.connect(lambda: self._browse_into(self.le_graxpert))
-        btn_ccl   = QPushButton("Browse…"); btn_ccl.clicked.connect(lambda: self._browse_dir(self.le_cosmic))
-        btn_star  = QPushButton("Browse…"); btn_star.clicked.connect(lambda: self._browse_into(self.le_starnet))
-        btn_astap = QPushButton("Browse…"); btn_astap.clicked.connect(lambda: self._browse_into(self.le_astap))
+        btn_grax  = QPushButton(self.tr("Browse…")); btn_grax.clicked.connect(lambda: self._browse_into(self.le_graxpert))
+        btn_ccl   = QPushButton(self.tr("Browse…")); btn_ccl.clicked.connect(lambda: self._browse_dir(self.le_cosmic))
+        btn_star  = QPushButton(self.tr("Browse…")); btn_star.clicked.connect(lambda: self._browse_into(self.le_starnet))
+        btn_astap = QPushButton(self.tr("Browse…")); btn_astap.clicked.connect(lambda: self._browse_into(self.le_astap))
 
         row_grax  = QHBoxLayout(); row_grax.addWidget(self.le_graxpert); row_grax.addWidget(btn_grax)
         row_ccl   = QHBoxLayout(); row_ccl.addWidget(self.le_cosmic);   row_ccl.addWidget(btn_ccl)
@@ -96,7 +86,6 @@ class SettingsDialog(QDialog):
 
         self.le_astrometry = QLineEdit()
         self.le_astrometry.setEchoMode(QLineEdit.EchoMode.Password)
-        self.le_astrometry.setText(self.settings.value("api/astrometry_key", "", type=str))
 
         # ---- WIMS defaults ----
         self.sp_lat = QDoubleSpinBox();  self.sp_lat.setRange(-90.0, 90.0);       self.sp_lat.setDecimals(6)
@@ -107,29 +96,14 @@ class SettingsDialog(QDialog):
         self.sp_min_alt = QDoubleSpinBox(); self.sp_min_alt.setRange(0.0, 90.0);  self.sp_min_alt.setDecimals(1)
         self.sp_obj_limit = QSpinBox(); self.sp_obj_limit.setRange(1, 1000)
 
-        self.sp_lat.setValue(self.settings.value("latitude", 0.0, type=float))
-        self.sp_lon.setValue(self.settings.value("longitude", 0.0, type=float))
-        self.le_date.setText(self.settings.value("date", "", type=str) or "")
-        self.le_time.setText(self.settings.value("time", "", type=str) or "")
-        tz_val = self.settings.value("timezone", "UTC", type=str) or "UTC"
-        idx = max(0, self.cb_tz.findText(tz_val)); self.cb_tz.setCurrentIndex(idx)
-        self.sp_min_alt.setValue(self.settings.value("min_altitude", 0.0, type=float))
-        self.sp_obj_limit.setValue(self.settings.value("object_limit", 100, type=int))
-
-        self.chk_autostretch_16bit = QCheckBox("High-quality autostretch (16-bit; better gradients)")
-        self.chk_autostretch_16bit.setToolTip("Compute autostretch on a 16-bit histogram (smoother gradients).")
-        self.chk_autostretch_16bit.setChecked(
-            self.settings.value("display/autostretch_16bit", True, type=bool)
-        )
+        self.chk_autostretch_16bit = QCheckBox(self.tr("High-quality autostretch (16-bit; better gradients)"))
+        self.chk_autostretch_16bit.setToolTip(self.tr("Compute autostretch on a 16-bit histogram (smoother gradients)."))
 
         self.slider_bg_opacity = QSlider(Qt.Orientation.Horizontal)
         self.slider_bg_opacity.setRange(0, 100)
-        current_opacity = self.settings.value("display/bg_opacity", 50, type=int)
-        self.slider_bg_opacity.setValue(current_opacity)
-        # Keep a copy of the original opacity so we can restore it if the user cancels
-        self._initial_bg_opacity = int(current_opacity)
+        self._initial_bg_opacity = 50
 
-        self.lbl_bg_opacity_val = QLabel(f"{current_opacity}%")
+        self.lbl_bg_opacity_val = QLabel("50%")
         self.lbl_bg_opacity_val.setFixedWidth(40)
 
         def _on_opacity_changed(val):
@@ -153,14 +127,13 @@ class SettingsDialog(QDialog):
         # ---- Custom background: choose/clear preview ----
         self.le_bg_path = QLineEdit()
         self.le_bg_path.setReadOnly(True)
-        # remember initial custom background so Cancel can restore it
-        self._initial_bg_path = self.settings.value("ui/custom_background", "", type=str) or ""
-        self.le_bg_path.setText(self._initial_bg_path)
-        btn_choose_bg = QPushButton("Choose Background…")
-        btn_choose_bg.setToolTip("Pick a PNG or JPG to use as the application background")
+        self._initial_bg_path = ""
+        
+        btn_choose_bg = QPushButton(self.tr("Choose Background…"))
+        btn_choose_bg.setToolTip(self.tr("Pick a PNG or JPG to use as the application background"))
         btn_choose_bg.clicked.connect(self._choose_background_clicked)
-        btn_clear_bg = QPushButton("Clear")
-        btn_clear_bg.setToolTip("Remove custom background and restore default")
+        btn_clear_bg = QPushButton(self.tr("Clear"))
+        btn_clear_bg.setToolTip(self.tr("Remove custom background and restore default"))
         btn_clear_bg.clicked.connect(self._clear_background_clicked)
 
         row_bg_image = QHBoxLayout()
@@ -188,57 +161,49 @@ class SettingsDialog(QDialog):
         cols.addLayout(right_col, 1)
 
         # ---- Left column: Paths & Integrations ----
-        left_col.addRow(QLabel("<b>Paths & Integrations</b>"))
-        w = QWidget(); w.setLayout(row_grax);  left_col.addRow("GraXpert executable:", w)
-        w = QWidget(); w.setLayout(row_ccl);   left_col.addRow("Cosmic Clarity folder:", w)
-        w = QWidget(); w.setLayout(row_star);  left_col.addRow("StarNet executable:", w)
-        w = QWidget(); w.setLayout(row_astap); left_col.addRow("ASTAP executable:", w)
-        left_col.addRow("Astrometry.net API key:", self.le_astrometry)
+        left_col.addRow(QLabel(self.tr("<b>Paths & Integrations</b>")))
+        w = QWidget(); w.setLayout(row_grax);  left_col.addRow(self.tr("GraXpert executable:"), w)
+        w = QWidget(); w.setLayout(row_ccl);   left_col.addRow(self.tr("Cosmic Clarity folder:"), w)
+        w = QWidget(); w.setLayout(row_star);  left_col.addRow(self.tr("StarNet executable:"), w)
+        w = QWidget(); w.setLayout(row_astap); left_col.addRow(self.tr("ASTAP executable:"), w)
+        left_col.addRow(self.tr("Astrometry.net API key:"), self.le_astrometry)
         left_col.addRow(self.chk_save_shortcuts)
         row_theme = QHBoxLayout()
         row_theme.addWidget(self.cb_theme, 1)
         row_theme.addWidget(self.btn_theme_custom)
         w_theme = QWidget()
         w_theme.setLayout(row_theme)
-        left_col.addRow("Theme:", w_theme)
+        left_col.addRow(self.tr("Theme:"), w_theme)
+        left_col.addRow(self.tr("Language:"), self.cb_language)
 
         # ---- Display (moved under Theme) ----
-        left_col.addRow(QLabel("<b>Display</b>"))
+        left_col.addRow(QLabel(self.tr("<b>Display</b>")))
         left_col.addRow(self.chk_autostretch_16bit)
-        left_col.addRow("Background Opacity:", w_bg_opacity)
-        left_col.addRow("Background Image:", w_bg_image)
+        left_col.addRow(self.tr("Background Opacity:"), w_bg_opacity)
+        left_col.addRow(self.tr("Background Image:"), w_bg_image)
 
         # ---- Right column: WIMS + RA/Dec + Updates + Display ----
-        right_col.addRow(QLabel("<b>What's In My Sky — Defaults</b>"))
-        right_col.addRow("Latitude (°):", self.sp_lat)
-        right_col.addRow("Longitude (°):", self.sp_lon)
-        right_col.addRow("Date (YYYY-MM-DD):", self.le_date)
-        right_col.addRow("Time (HH:MM):", self.le_time)
-        right_col.addRow("Time Zone:", self.cb_tz)
-        right_col.addRow("Min Altitude (°):", self.sp_min_alt)
-        right_col.addRow("Object Limit:", self.sp_obj_limit)
+        right_col.addRow(QLabel(self.tr("<b>What's In My Sky — Defaults</b>")))
+        right_col.addRow(self.tr("Latitude (°):"), self.sp_lat)
+        right_col.addRow(self.tr("Longitude (°):"), self.sp_lon)
+        right_col.addRow(self.tr("Date (YYYY-MM-DD):"), self.le_date)
+        right_col.addRow(self.tr("Time (HH:MM):"), self.le_time)
+        right_col.addRow(self.tr("Time Zone:"), self.cb_tz)
+        right_col.addRow(self.tr("Min Altitude (°):"), self.sp_min_alt)
+        right_col.addRow(self.tr("Object Limit:"), self.sp_obj_limit)
 
         # ---- RA/Dec Overlay ----
-        right_col.addRow(QLabel("<b>RA/Dec Overlay</b>"))
-        self.chk_wcs_enabled = QCheckBox("Show RA/Dec grid")
-        self.chk_wcs_enabled.setChecked(self.settings.value("wcs_grid/enabled", True, type=bool))
+        right_col.addRow(QLabel(self.tr("<b>RA/Dec Overlay</b>")))
+        self.chk_wcs_enabled = QCheckBox(self.tr("Show RA/Dec grid"))
+        
         right_col.addRow(self.chk_wcs_enabled)
 
         self.cb_wcs_mode = QComboBox(); self.cb_wcs_mode.addItems(["Auto", "Fixed spacing"])
-        self.cb_wcs_mode.setCurrentIndex(
-            0 if (self.settings.value("wcs_grid/mode", "auto", type=str) == "auto") else 1
-        )
-
         self.cb_wcs_unit = QComboBox(); self.cb_wcs_unit.addItems(["deg", "arcmin"])
-        self.cb_wcs_unit.setCurrentIndex(
-            0 if (self.settings.value("wcs_grid/step_unit", "deg", type=str) == "deg") else 1
-        )
-
+        
         self.sp_wcs_step = QDoubleSpinBox()
         self.sp_wcs_step.setDecimals(3); self.sp_wcs_step.setRange(0.001, 90.0)
-        self.sp_wcs_step.setValue(self.settings.value("wcs_grid/step_value", 1.0, type=float))
-        self.sp_wcs_step.setEnabled(self.cb_wcs_mode.currentIndex() == 1)
-
+        
         def _sync_suffix():
             self.sp_wcs_step.setSuffix(" °" if self.cb_wcs_unit.currentIndex() == 0 else " arcmin")
         _sync_suffix()
@@ -246,20 +211,18 @@ class SettingsDialog(QDialog):
         self.cb_wcs_mode.currentIndexChanged.connect(lambda i: self.sp_wcs_step.setEnabled(i == 1))
 
         row_wcs = QHBoxLayout()
-        row_wcs.addWidget(QLabel("Mode:")); row_wcs.addWidget(self.cb_wcs_mode)
+        row_wcs.addWidget(QLabel(self.tr("Mode:"))); row_wcs.addWidget(self.cb_wcs_mode)
         row_wcs.addSpacing(8)
-        row_wcs.addWidget(QLabel("Step:")); row_wcs.addWidget(self.sp_wcs_step, 1); row_wcs.addWidget(self.cb_wcs_unit)
+        row_wcs.addWidget(QLabel(self.tr("Step:"))); row_wcs.addWidget(self.sp_wcs_step, 1); row_wcs.addWidget(self.cb_wcs_unit)
         _w = QWidget(); _w.setLayout(row_wcs)
         right_col.addRow(_w)
 
         # ---- Updates ----
-        right_col.addRow(QLabel("<b>Updates</b>"))
+        right_col.addRow(QLabel(self.tr("<b>Updates</b>")))
         right_col.addRow(self.chk_updates_startup)
         w = QWidget(); w.setLayout(row_updates_url)
-        right_col.addRow("Updates JSON URL:", w)
+        right_col.addRow(self.tr("Updates JSON URL:"), w)
 
-        
-        
         # ---- Buttons ----
         btns = QDialogButtonBox(
             QDialogButtonBox.StandardButton.Ok | QDialogButtonBox.StandardButton.Cancel, parent=self
@@ -268,6 +231,105 @@ class SettingsDialog(QDialog):
         btns.rejected.connect(self.reject)
         root.addWidget(btns)
 
+        # Initial Load:
+        self.refresh_ui()
+
+    def refresh_ui(self):
+        """
+        Reloads all settings from self.settings and updates the UI widgets.
+        Call this before showing the cached dialog to ensure it matches current state.
+        """
+        # Updates
+        self.chk_updates_startup.setChecked(
+            self.settings.value("updates/check_on_startup", True, type=bool)
+        )
+        self.le_updates_url.setText(
+            self.settings.value(
+                "updates/url",
+                "https://raw.githubusercontent.com/setiastro/setiastrosuitepro/main/updates.json",
+                type=str
+            )
+        )
+        
+        # Shortcuts
+        self.chk_save_shortcuts.setChecked(
+            self.settings.value("shortcuts/save_on_exit", True, type=bool)
+        )
+        
+        # Theme
+        theme_val = (self.settings.value("ui/theme", "system", type=str) or "system").lower()
+        index_map = {"dark": 0, "gray": 1, "light": 2, "system": 3, "custom": 4}
+        self.cb_theme.setCurrentIndex(index_map.get(theme_val, 2))
+        self.btn_theme_custom.setEnabled(theme_val == "custom")
+        
+        # Language
+        current_lang = get_saved_language()
+        try:
+            lang_idx = self._lang_codes.index(current_lang)
+        except ValueError:
+            lang_idx = 0
+            # fallback to whatever is first or default
+            if "en" in self._lang_codes:
+                lang_idx = self._lang_codes.index("en")
+        
+        self.cb_language.blockSignals(True)
+        self.cb_language.setCurrentIndex(lang_idx)
+        self.cb_language.blockSignals(False)
+        self._initial_language = current_lang  # Track for restart notification
+        
+        # Path fields
+        self.le_graxpert.setText(self.settings.value("paths/graxpert", "", type=str))
+        self.le_cosmic.setText(self.settings.value("paths/cosmic_clarity", "", type=str))
+        self.le_starnet.setText(self.settings.value("paths/starnet", "", type=str))
+        self.le_astap.setText(self.settings.value("paths/astap", "", type=str))
+        self.le_astrometry.setText(self.settings.value("api/astrometry_key", "", type=str))
+
+        # WIMS
+        self.sp_lat.setValue(self.settings.value("latitude", 0.0, type=float))
+        self.sp_lon.setValue(self.settings.value("longitude", 0.0, type=float))
+        self.le_date.setText(self.settings.value("date", "", type=str) or "")
+        self.le_time.setText(self.settings.value("time", "", type=str) or "")
+        tz_val = self.settings.value("timezone", "UTC", type=str) or "UTC"
+        idx = max(0, self.cb_tz.findText(tz_val))
+        self.cb_tz.setCurrentIndex(idx)
+        self.sp_min_alt.setValue(self.settings.value("min_altitude", 0.0, type=float))
+        self.sp_obj_limit.setValue(self.settings.value("object_limit", 100, type=int))
+        
+        # Display
+        self.chk_autostretch_16bit.setChecked(
+            self.settings.value("display/autostretch_16bit", True, type=bool)
+        )
+        
+        current_opacity = self.settings.value("display/bg_opacity", 50, type=int)
+        self.slider_bg_opacity.blockSignals(True)
+        self.slider_bg_opacity.setValue(current_opacity)
+        self.slider_bg_opacity.blockSignals(False)
+        self.lbl_bg_opacity_val.setText(f"{current_opacity}%")
+        self._initial_bg_opacity = int(current_opacity) # For cancel/revert
+        
+        # Custom background
+        self._initial_bg_path = self.settings.value("ui/custom_background", "", type=str) or ""
+        self.le_bg_path.setText(self._initial_bg_path)
+        
+        # RA/Dec Overlay
+        self.chk_wcs_enabled.setChecked(self.settings.value("wcs_grid/enabled", True, type=bool))
+        
+        self.cb_wcs_mode.blockSignals(True)
+        self.cb_wcs_mode.setCurrentIndex(
+            0 if (self.settings.value("wcs_grid/mode", "auto", type=str) == "auto") else 1
+        )
+        self.cb_wcs_mode.blockSignals(False)
+        
+        self.cb_wcs_unit.blockSignals(True)
+        self.cb_wcs_unit.setCurrentIndex(
+            0 if (self.settings.value("wcs_grid/step_unit", "deg", type=str) == "deg") else 1
+        )
+        self.cb_wcs_unit.blockSignals(False)
+        
+        self.sp_wcs_step.setValue(self.settings.value("wcs_grid/step_value", 1.0, type=float))
+        self.sp_wcs_step.setEnabled(self.cb_wcs_mode.currentIndex() == 1)
+        self.sp_wcs_step.setSuffix(" °" if self.cb_wcs_unit.currentIndex() == 0 else " arcmin")
+
     def reject(self):
         """User cancelled: restore the original background opacity (revert live changes)."""
         try:
@@ -430,6 +492,21 @@ class SettingsDialog(QDialog):
             theme_val = "custom"
         self.settings.setValue("ui/theme", theme_val)
 
+        # Language
+        lang_idx = self.cb_language.currentIndex()
+        new_lang = self._lang_codes[lang_idx] if 0 <= lang_idx < len(self._lang_codes) else "en"
+        save_language(new_lang)
+        
+        # Apply language change immediately if changed
+        if new_lang != self._initial_language:
+            from PyQt6.QtWidgets import QMessageBox
+            
+            QMessageBox.information(
+                self,
+                self.tr("Restart required"),
+                self.tr("Language changed. Please manually restart the application to apply the new language.")
+            )
+
         self.settings.sync()
 
         # Apply now if the parent knows how

setiastro/saspro/perfect_palette_picker.py

@@ -28,6 +28,7 @@ class PaletteAdjustDialog(QDialog):
         super().__init__(owner)
         self.setWindowTitle("Adjust Palette Intensities")
         self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
         self.setModal(False)
         #self.setAttribute(Qt.WidgetAttribute.WA_DeleteOnClose, True)
 
@@ -53,10 +54,21 @@ class PaletteAdjustDialog(QDialog):
 
         # Zoom controls
         zoom_layout = QHBoxLayout()
-        btn_zoom_in  = themed_toolbtn("zoom-in", "Zoom In")
-        btn_zoom_out = themed_toolbtn("zoom-out", "Zoom Out")
-        btn_fit      = themed_toolbtn("zoom-fit-best", "Fit to Preview")
-        zoom_layout.addWidget(btn_zoom_in); zoom_layout.addWidget(btn_zoom_out); zoom_layout.addWidget(btn_fit)
+
+        self.btn_zoom_in  = themed_toolbtn("zoom-in", "Zoom In")
+        self.btn_zoom_out = themed_toolbtn("zoom-out", "Zoom Out")
+        self.btn_fit      = themed_toolbtn("zoom-fit-best", "Fit to Preview")
+
+        self.btn_zoom_in.clicked.connect(lambda: self._change_zoom(1.25))
+        self.btn_zoom_out.clicked.connect(lambda: self._change_zoom(0.8))
+        self.btn_fit.clicked.connect(self._fit_to_preview)
+
+        zoom_layout.addStretch(1)
+        zoom_layout.addWidget(self.btn_zoom_out)
+        zoom_layout.addWidget(self.btn_zoom_in)
+        zoom_layout.addWidget(self.btn_fit)
+        zoom_layout.addStretch(1)
+
         vlayout.addLayout(zoom_layout)
 
         # Preview
@@ -136,11 +148,17 @@ class PaletteAdjustDialog(QDialog):
         self._rescale_pixmap()
 
     def _fit_to_preview(self):
-        if not hasattr(self, "_base_pixmap"): return
-        vp_w = self.preview_area.viewport().width()
-        self.zoom_factor = vp_w / max(1, self._base_pixmap.width())
+        if not hasattr(self, "_base_pixmap"):
+            return
+        vp = self.preview_area.viewport().size()
+        pm = self._base_pixmap.size()
+        if pm.width() <= 0 or pm.height() <= 0:
+            return
+        k = min(vp.width() / pm.width(), vp.height() / pm.height())
+        self.zoom_factor = max(0.1, min(10.0, k))
         self._rescale_pixmap()
 
+
     def _on_reset(self):
         for s in (self._ha_slider, self._oiii_slider, self._sii_slider):
             s.setValue(100)