Skip to content

bug-fix-missing horizontal scroll bar at the preview data and app bre… #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Ebenezer5542 wants to merge 2 commits into
base: feature/new-gui
Choose a base branch
from
Open

bug-fix-missing horizontal scroll bar at the preview data and app bre… #22

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
af8ea15
bug-fix-missing horizontal scroll bar at the preview data and app bre…
Ebenezer5542 Nov 28, 2025
d751eec
Clean up debug comments
Ebenezer5542 Dec 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 13 additions & 6 deletions src/pii_detector/gui/flet_app/ui/screens/progress.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -349,9 +349,9 @@ def _update_current_task(self, task: str):

def start_analysis(self):
"""Start the PII detection analysis."""
# print("DEBUG: start_analysis() called")
print("DEBUG: start_analysis() called")
if not self.is_analysis_running:
# print("DEBUG: Analysis is not running, starting new analysis")
print("DEBUG: Analysis is not running, starting new analysis")
self.is_analysis_running = True
self.analysis_cancelled = False
self.analysis_complete = False
Expand Down Expand Up @@ -529,22 +529,24 @@ def close_dialog(e):
self.page.open(dialog)

def on_state_changed(self, state: AppState):
"""Handle state changes."""
# Only start analysis when we actually navigate TO the progress screen
"Handle state changes."
# Always start a new analysis if not currently running
if (
state.current_screen == AppConstants.SCREEN_PROGRESS
and not self.is_analysis_running
and not self.analysis_complete
):
# Small delay to ensure UI is fully loaded
# Reset completed flag before starting a new run
self.analysis_complete = False
threading.Timer(0.5, self.start_analysis).start()

def on_screen_enter(self):
"""Enter this screen and reset analysis state."""
# print("DEBUG: on_screen_enter() called")
# Reset analysis state for new analysis
self.is_analysis_running = False
self.analysis_cancelled = False
self.analysis_complete = False
print("DEBUG: Analysis state reset")

if self.overall_progress_bar:
self.overall_progress_bar.value = 0
Expand All @@ -567,6 +569,11 @@ def on_screen_enter(self):

# Reset progress messages for copying
self.progress_messages = ["Ready to start analysis"]
print("DEBUG: UI components reset")

# Reset background processor
self.background_processor = BackgroundProcessor(self.adapter)
print("DEBUG: BackgroundProcessor recreated")

def _initialize_default_anonymization_methods(self, results):
"""Initialize smart default anonymization methods for detected PII columns.
Expand Down
5 changes: 4 additions & 1 deletion src/pii_detector/gui/flet_app/ui/screens/results.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,10 @@ def close_dialog(e):
border_radius=4,
),
ft.Container(
content=preview_table,
content=ft.Row(
controls=[preview_table],
scroll=ft.ScrollMode.AUTO,
),
width=800,
),
],
Expand Down