-
Notifications
You must be signed in to change notification settings - Fork 35
Use autolayout for loading indicator #1628
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.
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
Conversation
There was a problem hiding this comment.
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 introduces autolayout support for the loading indicator in the macOS webview UI controller, controlled by a feature flag. The changes aim to modernize the layout approach from frame-based positioning to constraint-based autolayout.
- Adds a new feature flag
MSID_FLIGHT_USE_AUTOLAYOUT_FOR_LOADING_INDICATORto enable/disable autolayout behavior - Replaces magic number
32with named constantMSID_LOADING_INDICATOR_SIZEfor better maintainability - Implements autolayout constraints to center the loading indicator in the webview
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| IdentityCore/src/MSIDConstants.h | Declares the new feature flag constant for autolayout control |
| IdentityCore/src/MSIDConstants.m | Defines the feature flag constant with its string value |
| IdentityCore/src/webview/embeddedWebview/ui/mac/MSIDWebviewUIController.m | Adds autolayout implementation for the loading indicator, defines size constant, and replaces magic numbers |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
IdentityCore/src/webview/embeddedWebview/ui/mac/MSIDWebviewUIController.m
Show resolved
Hide resolved
IdentityCore/src/webview/embeddedWebview/ui/mac/MSIDWebviewUIController.m
Outdated
Show resolved
Hide resolved
IdentityCore/src/webview/embeddedWebview/ui/mac/MSIDWebviewUIController.m
Show resolved
Hide resolved
kaisong1990
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor question
Proposed changes
Use autolayout for loading indicator
Type of change
Risk
Additional information