MoleditPy 2.7.2__tar.gz → 2.8.0__tar.gz

{moleditpy-2.7.2 → moleditpy-2.8.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: MoleditPy
-Version: 2.7.2
+Version: 2.8.0
 Summary: A cross-platform, simple, and intuitive molecular structure editor built in Python. It allows 2D molecular drawing and 3D structure visualization. It supports exporting structure files for input to DFT calculation software.
 Author-email: HiroYokoyama <titech.yoko.hiro@gmail.com>
 License:                     GNU GENERAL PUBLIC LICENSE
@@ -694,7 +694,8 @@ Requires-Python: <3.15,>=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: numpy
-Requires-Dist: pyqt6<6.11
+Requires-Dist: pyqt6<6.10; sys_platform == "darwin"
+Requires-Dist: pyqt6<6.11; sys_platform != "darwin"
 Requires-Dist: pyvista<0.48
 Requires-Dist: pyvistaqt<0.12
 Requires-Dist: rdkit<2025.10
@@ -712,7 +713,7 @@ Dynamic: license-file
 ![Core Logic Coverage](https://img.shields.io/badge/core_logic_coverage-73%25-green)
 ![Overall Coverage](https://img.shields.io/badge/coverage-50%25-orange)
 ![GUI Status](https://img.shields.io/badge/GUI-Manually_Verified-blue)
-![Pylint Score](https://img.shields.io/badge/pylint-8.67%2F10-brightgreen)
+![Pylint Score](https://img.shields.io/badge/pylint-8.63%2F10-brightgreen)
 [![PyPI Downloads](https://static.pepy.tech/personalized-badge/moleditpy?period=total&units=INTERNATIONAL_SYSTEM&left_color=BLACK&right_color=GREEN&left_text=downloads)](https://pepy.tech/projects/moleditpy)
 
 [🇯🇵 日本語 (Japanese)](#japanese)

{moleditpy-2.7.2 → moleditpy-2.8.0}/README.md

@@ -9,7 +9,7 @@
 ![Core Logic Coverage](https://img.shields.io/badge/core_logic_coverage-73%25-green)
 ![Overall Coverage](https://img.shields.io/badge/coverage-50%25-orange)
 ![GUI Status](https://img.shields.io/badge/GUI-Manually_Verified-blue)
-![Pylint Score](https://img.shields.io/badge/pylint-8.67%2F10-brightgreen)
+![Pylint Score](https://img.shields.io/badge/pylint-8.63%2F10-brightgreen)
 [![PyPI Downloads](https://static.pepy.tech/personalized-badge/moleditpy?period=total&units=INTERNATIONAL_SYSTEM&left_color=BLACK&right_color=GREEN&left_text=downloads)](https://pepy.tech/projects/moleditpy)
 
 [🇯🇵 日本語 (Japanese)](#japanese)

{moleditpy-2.7.2 → moleditpy-2.8.0}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 name = "MoleditPy"
 
-version = "2.7.2"
+version = "2.8.0"
 
 license = {file = "LICENSE"}
 
@@ -34,7 +34,8 @@ classifiers = [
 
 dependencies = [
   "numpy",
-  "pyqt6 < 6.11",
+  "pyqt6 < 6.10; sys_platform == 'darwin'",
+  "pyqt6 < 6.11; sys_platform != 'darwin'",
   "pyvista < 0.48",
   "pyvistaqt < 0.12",
   "rdkit < 2025.10",

{moleditpy-2.7.2 → moleditpy-2.8.0}/src/MoleditPy.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: MoleditPy
-Version: 2.7.2
+Version: 2.8.0
 Summary: A cross-platform, simple, and intuitive molecular structure editor built in Python. It allows 2D molecular drawing and 3D structure visualization. It supports exporting structure files for input to DFT calculation software.
 Author-email: HiroYokoyama <titech.yoko.hiro@gmail.com>
 License:                     GNU GENERAL PUBLIC LICENSE
@@ -694,7 +694,8 @@ Requires-Python: <3.15,>=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: numpy
-Requires-Dist: pyqt6<6.11
+Requires-Dist: pyqt6<6.10; sys_platform == "darwin"
+Requires-Dist: pyqt6<6.11; sys_platform != "darwin"
 Requires-Dist: pyvista<0.48
 Requires-Dist: pyvistaqt<0.12
 Requires-Dist: rdkit<2025.10
@@ -712,7 +713,7 @@ Dynamic: license-file
 ![Core Logic Coverage](https://img.shields.io/badge/core_logic_coverage-73%25-green)
 ![Overall Coverage](https://img.shields.io/badge/coverage-50%25-orange)
 ![GUI Status](https://img.shields.io/badge/GUI-Manually_Verified-blue)
-![Pylint Score](https://img.shields.io/badge/pylint-8.67%2F10-brightgreen)
+![Pylint Score](https://img.shields.io/badge/pylint-8.63%2F10-brightgreen)
 [![PyPI Downloads](https://static.pepy.tech/personalized-badge/moleditpy?period=total&units=INTERNATIONAL_SYSTEM&left_color=BLACK&right_color=GREEN&left_text=downloads)](https://pepy.tech/projects/moleditpy)
 
 [🇯🇵 日本語 (Japanese)](#japanese)

{moleditpy-2.7.2 → moleditpy-2.8.0}/src/MoleditPy.egg-info/requires.txt

@@ -1,6 +1,11 @@
 numpy
-pyqt6<6.11
 pyvista<0.48
 pyvistaqt<0.12
 rdkit<2025.10
 openbabel-wheel<3.2
+
+[:sys_platform != "darwin"]
+pyqt6<6.11
+
+[:sys_platform == "darwin"]
+pyqt6<6.10

{moleditpy-2.7.2 → moleditpy-2.8.0}/src/moleditpy/modules/angle_dialog.py

@@ -17,16 +17,17 @@ from PyQt6.QtWidgets import (
     QLineEdit,
     QPushButton,
     QRadioButton,
+    QSlider,
     QVBoxLayout,
     QWidget,
 )
 
 try:
     from .dialog3_d_picking_mixin import Dialog3DPickingMixin
-    from .mol_geometry import get_connected_group
+    from .mol_geometry import adjust_bond_angle, get_connected_group, rodrigues_rotate
 except Exception:
     from modules.dialog3_d_picking_mixin import Dialog3DPickingMixin
-    from modules.mol_geometry import get_connected_group
+    from modules.mol_geometry import adjust_bond_angle, get_connected_group, rodrigues_rotate
 
 import numpy as np
 from PyQt6.QtCore import Qt
@@ -76,8 +77,23 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
         angle_layout.addWidget(QLabel("New angle (degrees):"))
         self.angle_input = QLineEdit()
         self.angle_input.setPlaceholderText("109.5")
+        self.angle_input.textChanged.connect(self.on_angle_input_changed)
         angle_layout.addWidget(self.angle_input)
+
+        self.angle_slider = QSlider(Qt.Orientation.Horizontal)
+        self.angle_slider.setMinimum(-180)
+        self.angle_slider.setMaximum(180)
+        self.angle_slider.setValue(109)
+        self.angle_slider.setTickPosition(QSlider.TickPosition.TicksBelow)
+        self.angle_slider.setTickInterval(45)
+        self.angle_slider.setEnabled(False)
+        self.angle_slider.sliderPressed.connect(self.on_slider_pressed)
+        self.angle_slider.sliderMoved.connect(self.on_slider_moved)
+        self.angle_slider.sliderReleased.connect(self.on_slider_released)
+        self.angle_slider.valueChanged.connect(self.on_slider_value_changed)
+        self._slider_dragging = False
         layout.addLayout(angle_layout)
+        layout.addWidget(self.angle_slider)
 
         # Movement options
         group_box = QWidget()
@@ -201,7 +217,13 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.apply_button.setEnabled(False)
             # Clear angle input when no selection
             try:
+                self.angle_input.blockSignals(True)
                 self.angle_input.clear()
+                self.angle_input.blockSignals(False)
+                self.angle_slider.blockSignals(True)
+                self.angle_slider.setValue(109)
+                self.angle_slider.setEnabled(False)
+                self.angle_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -217,7 +239,13 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.add_selection_label(self.atom1_idx, "1")
             # Clear angle input while selection is incomplete
             try:
+                self.angle_input.blockSignals(True)
                 self.angle_input.clear()
+                self.angle_input.blockSignals(False)
+                self.angle_slider.blockSignals(True)
+                self.angle_slider.setValue(109)
+                self.angle_slider.setEnabled(False)
+                self.angle_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -235,7 +263,13 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.add_selection_label(self.atom2_idx, "2(vertex)")
             # Clear angle input while selection is incomplete
             try:
+                self.angle_input.blockSignals(True)
                 self.angle_input.clear()
+                self.angle_input.blockSignals(False)
+                self.angle_slider.blockSignals(True)
+                self.angle_slider.setValue(109)
+                self.angle_slider.setEnabled(False)
+                self.angle_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -253,7 +287,15 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.apply_button.setEnabled(True)
             # Update angle input box with current angle
             try:
+                self.angle_input.blockSignals(True)
                 self.angle_input.setText(f"{current_angle:.2f}")
+                self.angle_input.blockSignals(False)
+                self.angle_slider.blockSignals(True)
+                slider_val = int(round(current_angle))
+                slider_val = max(-180, min(180, slider_val))
+                self.angle_slider.setValue(slider_val)
+                self.angle_slider.setEnabled(True)
+                self.angle_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -278,18 +320,73 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
         angle_rad = np.arccos(cos_angle)
         return np.degrees(angle_rad)
 
+    def on_angle_input_changed(self, text):
+        """Line edit text changed, update slider."""
+        if not self.angle_input.isEnabled() or not self.apply_button.isEnabled():
+            return
+        try:
+            val = float(text)
+            wrapped_val = (val + 180) % 360 - 180
+            self.angle_slider.blockSignals(True)
+            self.angle_slider.setValue(int(round(wrapped_val)))
+            self.angle_slider.blockSignals(False)
+        except ValueError:
+            pass
+
+    def on_slider_pressed(self):
+        """Remember the state before slider dragging starts."""
+        if self.atom1_idx is None or self.atom2_idx is None or self.atom3_idx is None:
+            return
+        self._slider_dragging = True
+        self.main_window.push_undo_state()
+        # Snapshot positions so the rotation axis stays stable during drag
+        self._snapshot_positions = self.mol.GetConformer().GetPositions().copy()
+
+    def on_slider_moved(self, value):
+        """Update geometry in real-time while dragging."""
+        if self.atom1_idx is None or self.atom2_idx is None or self.atom3_idx is None:
+            return
+        
+        self.angle_input.blockSignals(True)
+        self.angle_input.setText(f"{value}")
+        self.angle_input.blockSignals(False)
+        
+        self.adjust_angle(float(value))
+
+    def on_slider_released(self):
+        """Finalize slider dragging."""
+        self._slider_dragging = False
+        self._snapshot_positions = None
+        self.main_window.draw_molecule_3d(self.mol)
+        self.main_window.update_chiral_labels()
+
+    def on_slider_value_changed(self, value):
+        """Handle click-to-position on the slider track."""
+        if self._slider_dragging:
+            return
+        if self.atom1_idx is None or self.atom2_idx is None or self.atom3_idx is None:
+            return
+        self.main_window.push_undo_state()
+        self.angle_input.blockSignals(True)
+        self.angle_input.setText(f"{value}")
+        self.angle_input.blockSignals(False)
+        self.adjust_angle(float(value))
+        self.main_window.update_chiral_labels()
+
     def apply_changes(self):
         """変更を適用"""
         if self.atom1_idx is None or self.atom2_idx is None or self.atom3_idx is None:
             return
 
         try:
-            new_angle = float(self.angle_input.text())
-            if new_angle < 0 or new_angle >= 360:
-                QMessageBox.warning(
-                    self, "Invalid Input", "Angle must be between 0 and 360 degrees."
-                )
-                return
+            raw_angle = float(self.angle_input.text())
+            # Automatic Range Wrapping
+            new_angle = (raw_angle + 180) % 360 - 180
+            
+            # Formally update the input to reflect wrapping
+            self.angle_input.blockSignals(True)
+            self.angle_input.setText(f"{new_angle:.2f}")
+            self.angle_input.blockSignals(False)
         except ValueError:
             QMessageBox.warning(self, "Invalid Input", "Please enter a valid number.")
             return
@@ -304,104 +401,76 @@ class AngleDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
         self.main_window.update_chiral_labels()
 
     def adjust_angle(self, new_angle_deg):
-        """角度を調整（均等回転オプション付き）"""
-        conf = self.mol.GetConformer()
-        pos1 = np.array(conf.GetAtomPosition(self.atom1_idx))
-        pos2 = np.array(conf.GetAtomPosition(self.atom2_idx))  # vertex
-        pos3 = np.array(conf.GetAtomPosition(self.atom3_idx))
-
-        vec1 = pos1 - pos2
-        vec2 = pos3 - pos2
-
-        # Current angle
-        current_angle_rad = np.arccos(
-            np.clip(
-                np.dot(vec1, vec2) / (np.linalg.norm(vec1) * np.linalg.norm(vec2)),
-                -1.0,
-                1.0,
-            )
-        )
-
-        # Target angle
-        target_angle_rad = np.radians(new_angle_deg)
+        """角度を調整（均等回転オプション付き）
 
-        # Rotation axis (perpendicular to the plane containing vec1 and vec2)
-        rotation_axis = np.cross(vec1, vec2)
-        rotation_axis_norm = np.linalg.norm(rotation_axis)
+        Uses the difference-based rotation approach via
+        :func:`~mol_geometry.adjust_bond_angle` to avoid 3D
+        rotational ambiguity.
 
-        if rotation_axis_norm == 0:
-            # Vectors are parallel, cannot rotate
-            return
+        During slider dragging, positions are restored from a snapshot
+        taken at press-time so that the rotation axis (cross product)
+        never flips direction.
+        """
+        conf = self.mol.GetConformer()
 
-        rotation_axis = rotation_axis / rotation_axis_norm
+        # Use snapshot if available (slider dragging) to keep the
+        # rotation axis stable; otherwise use current positions.
+        snapshot = getattr(self, '_snapshot_positions', None)
+        if snapshot is not None:
+            positions = snapshot.copy()
+        else:
+            positions = conf.GetPositions()  # N×3 ndarray (copy)
 
-        # Total rotation angle needed
-        total_rotation_angle = target_angle_rad - current_angle_rad
-
-        # Rodrigues' rotation formula
-        def rotate_vector(v, axis, angle):
-            cos_a = np.cos(angle)
-            sin_a = np.sin(angle)
-            return (
-                v * cos_a
-                + np.cross(axis, v) * sin_a
-                + axis * np.dot(axis, v) * (1 - cos_a)
-            )
+        idx_a = self.atom1_idx
+        idx_b = self.atom2_idx  # vertex
+        idx_c = self.atom3_idx
 
         if self.both_groups_radio.isChecked():
-            # Both arms rotate equally (half angle each in opposite directions)
-            half_rotation = total_rotation_angle / 2
+            # Both arms rotate equally (half angle each)
+            current_angle = self.calculate_angle()
+            half_delta_deg = (new_angle_deg - current_angle) / 2.0
+
+            group1 = get_connected_group(self.mol, idx_a, exclude=idx_b)
+            group3 = get_connected_group(self.mol, idx_c, exclude=idx_b)
 
-            # Get both connected groups
-            group1_atoms = get_connected_group(
-                self.mol, self.atom1_idx, exclude=self.atom2_idx
+            # Arm 1 rotates by −half (note: reversed A/C roles)
+            adjust_bond_angle(
+                positions, idx_c, idx_b, idx_a,
+                current_angle + half_delta_deg, group1,
             )
-            group3_atoms = get_connected_group(
-                self.mol, self.atom3_idx, exclude=self.atom2_idx
+            # Arm 3 rotates by +half
+            adjust_bond_angle(
+                positions, idx_a, idx_b, idx_c,
+                current_angle + half_delta_deg, group3,
             )
-
-            # Rotate group 1 by -half_rotation
-            for atom_idx in group1_atoms:
-                current_pos = np.array(conf.GetAtomPosition(atom_idx))
-                relative_pos = current_pos - pos2
-                rotated_pos = rotate_vector(relative_pos, rotation_axis, -half_rotation)
-                new_pos = pos2 + rotated_pos
-                conf.SetAtomPosition(atom_idx, new_pos.tolist())
-                self.main_window.atom_positions_3d[atom_idx] = new_pos
-
-            # Rotate group 3 by +half_rotation
-            for atom_idx in group3_atoms:
-                current_pos = np.array(conf.GetAtomPosition(atom_idx))
-                relative_pos = current_pos - pos2
-                rotated_pos = rotate_vector(relative_pos, rotation_axis, half_rotation)
-                new_pos = pos2 + rotated_pos
-                conf.SetAtomPosition(atom_idx, new_pos.tolist())
-                self.main_window.atom_positions_3d[atom_idx] = new_pos
-
         elif self.rotate_atom_radio.isChecked():
-            # Move only the third atom
-            new_vec2 = rotate_vector(vec2, rotation_axis, total_rotation_angle)
-            new_pos3 = pos2 + new_vec2
-            conf.SetAtomPosition(self.atom3_idx, new_pos3.tolist())
-            self.main_window.atom_positions_3d[self.atom3_idx] = new_pos3
+            # Move only atom C
+            adjust_bond_angle(
+                positions, idx_a, idx_b, idx_c,
+                new_angle_deg, {idx_c},
+            )
         else:
-            # Rotate the connected group around atom2 (vertex) - default behavior
+            # Default: rotate atom C and its connected sub-structure
             atoms_to_move = get_connected_group(
-                self.mol, self.atom3_idx, exclude=self.atom2_idx
+                self.mol, idx_c, exclude=idx_b,
+            )
+            adjust_bond_angle(
+                positions, idx_a, idx_b, idx_c,
+                new_angle_deg, atoms_to_move,
             )
 
-            for atom_idx in atoms_to_move:
-                current_pos = np.array(conf.GetAtomPosition(atom_idx))
-                # Transform to coordinate system centered at atom2
-                relative_pos = current_pos - pos2
-                # Rotate around the rotation axis
-                rotated_pos = rotate_vector(
-                    relative_pos, rotation_axis, total_rotation_angle
-                )
-                # Transform back to world coordinates
-                new_pos = pos2 + rotated_pos
-                conf.SetAtomPosition(atom_idx, new_pos.tolist())
-                self.main_window.atom_positions_3d[atom_idx] = new_pos
+        # Write updated positions back to the conformer and 3D cache
+        for i in range(conf.GetNumAtoms()):
+            conf.SetAtomPosition(i, positions[i].tolist())
+            self.main_window.atom_positions_3d[i] = positions[i]
 
         # Update the 3D view
         self.main_window.draw_molecule_3d(self.mol)
+
+    def reject(self):
+        super().reject()
+        try:
+            if self.main_window.current_mol:
+                self.main_window.draw_molecule_3d(self.main_window.current_mol)
+        except Exception:
+            pass

{moleditpy-2.7.2 → moleditpy-2.8.0}/src/moleditpy/modules/bond_length_dialog.py

@@ -20,6 +20,7 @@ from PyQt6.QtWidgets import (
     QMessageBox,
     QPushButton,
     QRadioButton,
+    QSlider,
     QVBoxLayout,
     QWidget,
 )
@@ -70,8 +71,23 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
         distance_layout.addWidget(QLabel("New distance (Å):"))
         self.distance_input = QLineEdit()
         self.distance_input.setPlaceholderText("1.54")
+        self.distance_input.textChanged.connect(self.on_distance_input_changed)
         distance_layout.addWidget(self.distance_input)
+
+        self.distance_slider = QSlider(Qt.Orientation.Horizontal)
+        self.distance_slider.setMinimum(10)   # 0.1 A
+        self.distance_slider.setMaximum(1000) # 10.0 A
+        self.distance_slider.setValue(154)    # 1.54 A
+        self.distance_slider.setTickPosition(QSlider.TickPosition.TicksBelow)
+        self.distance_slider.setTickInterval(100)
+        self.distance_slider.setEnabled(False)
+        self.distance_slider.sliderPressed.connect(self.on_slider_pressed)
+        self.distance_slider.sliderMoved.connect(self.on_slider_moved)
+        self.distance_slider.sliderReleased.connect(self.on_slider_released)
+        self.distance_slider.valueChanged.connect(self.on_slider_value_changed)
+        self._slider_dragging = False
         layout.addLayout(distance_layout)
+        layout.addWidget(self.distance_slider)
 
         # Movement options
         group_box = QWidget()
@@ -191,7 +207,13 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.apply_button.setEnabled(False)
             # Clear distance input when no selection
             try:
+                self.distance_input.blockSignals(True)
                 self.distance_input.clear()
+                self.distance_input.blockSignals(False)
+                self.distance_slider.blockSignals(True)
+                self.distance_slider.setValue(154)
+                self.distance_slider.setEnabled(False)
+                self.distance_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -207,7 +229,13 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.add_selection_label(self.atom1_idx, "1")
             # Clear distance input while selection is incomplete
             try:
+                self.distance_input.blockSignals(True)
                 self.distance_input.clear()
+                self.distance_input.blockSignals(False)
+                self.distance_slider.blockSignals(True)
+                self.distance_slider.setValue(154)
+                self.distance_slider.setEnabled(False)
+                self.distance_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -227,7 +255,15 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.apply_button.setEnabled(True)
             # Update the distance input box to show current distance
             try:
+                self.distance_input.blockSignals(True)
                 self.distance_input.setText(f"{current_distance:.3f}")
+                self.distance_input.blockSignals(False)
+                self.distance_slider.blockSignals(True)
+                slider_val = int(current_distance * 100)
+                slider_val = max(10, min(1000, slider_val))
+                self.distance_slider.setValue(slider_val)
+                self.distance_slider.setEnabled(True)
+                self.distance_slider.blockSignals(False)
             except Exception:  # pragma: no cover
                 import traceback
                 traceback.print_exc()
@@ -236,6 +272,58 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
             self.add_selection_label(self.atom1_idx, "1")
             self.add_selection_label(self.atom2_idx, "2")
 
+    def on_distance_input_changed(self, text):
+        """Line edit text changed, update slider."""
+        if not self.distance_input.isEnabled() or not self.apply_button.isEnabled():
+            return
+        try:
+            val = float(text)
+            if 0.1 <= val <= 10.0:
+                self.distance_slider.blockSignals(True)
+                self.distance_slider.setValue(int(val * 100))
+                self.distance_slider.blockSignals(False)
+        except ValueError:
+            pass
+
+    def on_slider_pressed(self):
+        """Remember the state before slider dragging starts."""
+        if self.atom1_idx is None or self.atom2_idx is None:
+            return
+        self._slider_dragging = True
+        self.main_window.push_undo_state()
+
+    def on_slider_moved(self, value):
+        """Update geometry in real-time while dragging."""
+        if self.atom1_idx is None or self.atom2_idx is None:
+            return
+        
+        new_distance = value / 100.0
+        self.distance_input.blockSignals(True)
+        self.distance_input.setText(f"{new_distance:.3f}")
+        self.distance_input.blockSignals(False)
+        
+        self.adjust_bond_length(new_distance)
+
+    def on_slider_released(self):
+        """Finalize slider dragging."""
+        self._slider_dragging = False
+        self.main_window.draw_molecule_3d(self.mol)
+        self.main_window.update_chiral_labels()
+
+    def on_slider_value_changed(self, value):
+        """Handle click-to-position on the slider track."""
+        if self._slider_dragging:
+            return  # Already handled by on_slider_moved
+        if self.atom1_idx is None or self.atom2_idx is None:
+            return
+        self.main_window.push_undo_state()
+        new_distance = value / 100.0
+        self.distance_input.blockSignals(True)
+        self.distance_input.setText(f"{new_distance:.3f}")
+        self.distance_input.blockSignals(False)
+        self.adjust_bond_length(new_distance)
+        self.main_window.update_chiral_labels()
+
     def apply_changes(self):
         """変更を適用"""
         if self.atom1_idx is None or self.atom2_idx is None:
@@ -330,3 +418,11 @@ class BondLengthDialog(Dialog3DPickingMixin, QDialog):  # pragma: no cover
 
         # Update the 3D view
         self.main_window.draw_molecule_3d(self.mol)
+
+    def reject(self):
+        super().reject()
+        try:
+            if self.main_window.current_mol:
+                self.main_window.draw_molecule_3d(self.main_window.current_mol)
+        except Exception:
+            pass