Skip to content

Fix UI window centering, python requirements and resource links broke…#56

Open
OskarYescas wants to merge 1 commit into
google:mainfrom
OskarYescas:ui-improvements
Open

Fix UI window centering, python requirements and resource links broke…#56
OskarYescas wants to merge 1 commit into
google:mainfrom
OskarYescas:ui-improvements

Conversation

@OskarYescas

Copy link
Copy Markdown

Based on customer feedback, this PR addresses Ui changes.

  1. Documentation Updates (README.md)
    Added an explicit note to the "Python Version" requirements section advising users to avoid Python Beta or Release Candidate (RC) versions, as they were observed causing installation errors for dependencies.

  2. Window Centering (ui/utils.py, migration_planner.py, ui/exchange_online_ui.py, ui/chats_ui.py)
    Created a lightweight center_window helper function in ui/utils.py.
    The main selector window and all three workload estimator windows (Exchange, Chat, Files) now consistently launch dead-center on the user's screen rather. Previously, this caused some misplacements if using different monitors.

  3. Resource Links Fix (ui/exchange_online_ui.py, ui/chats_ui.py)
    Fixed an issue where clicking the "Learn more" resource links did nothing.
    Imported the missing webbrowser standard library and successfully bound the webbrowser.open() action.
    Added cursor="hand2" to the link labels to provide immediate visual feedback that the text is clickable.
    Currerntly, the redirect links are the same (should be updated).

Testing
Successfully tested all three UI views locally on macOS.
Verified that links open the system's default browser correctly.
Verified that all screens center flawlessly upon launch.

@google-cla

google-cla Bot commented Jun 10, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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.

1 participant