Skip to content

Fix window height#788

Merged
amilcarlucas merged 2 commits into
masterfrom
fix-window-height
Sep 9, 2025
Merged

Fix window height#788
amilcarlucas merged 2 commits into
masterfrom
fix-window-height

Conversation

@amilcarlucas

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings September 9, 2025 13:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes window height calculations in two GUI windows to ensure proper display of interface elements.

  • Updates height calculation multiplier for dynamic content in directory selection window
  • Adjusts fixed height for connection selection window

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
ardupilot_methodic_configurator/frontend_tkinter_directory_selection.py Increases height multiplier from 21 to 23 pixels per setting for dynamic window sizing
ardupilot_methodic_configurator/frontend_tkinter_connection_selection.py Increases fixed window height from 450 to 462 pixels

@github-actions

github-actions Bot commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8227 6369 77% 73% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
ardupilot_methodic_configurator/frontend_tkinter_connection_selection.py 87% 🟢
ardupilot_methodic_configurator/frontend_tkinter_directory_selection.py 68% 🟢
TOTAL 77% 🟢

updated for commit: a63f397 by action🐍

@github-actions

github-actions Bot commented Sep 9, 2025

Copy link
Copy Markdown
Contributor

Test Results

    2 files      2 suites   1m 25s ⏱️
1 653 tests 1 652 ✅ 1 💤 0 ❌
3 306 runs  3 304 ✅ 2 💤 0 ❌

Results for commit a63f397.

@amilcarlucas amilcarlucas merged commit d7c45a2 into master Sep 9, 2025
28 checks passed
@amilcarlucas amilcarlucas deleted the fix-window-height branch September 9, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants