Skip to content

Conversation

@antrix1989
Copy link
Contributor

@antrix1989 antrix1989 commented Nov 27, 2025

Proposed changes

Use autolayout for loading indicator

Type of change

  • Feature work
  • Bug fix
  • Documentation
  • Engineering change
  • Test
  • Logging/Telemetry

Risk

  • High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
  • Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
  • Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information

Copilot finished reviewing on behalf of antrix1989 November 27, 2025 05:39
Copy link
Contributor

Copilot AI left a 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_INDICATOR to enable/disable autolayout behavior
  • Replaces magic number 32 with named constant MSID_LOADING_INDICATOR_SIZE for 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.

Copy link
Contributor

@kaisong1990 kaisong1990 left a comment

Choose a reason for hiding this comment

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

minor question

@antrix1989 antrix1989 enabled auto-merge December 1, 2025 19:33
@antrix1989 antrix1989 merged commit 655dfb8 into dev Dec 1, 2025
16 of 18 checks passed
@antrix1989 antrix1989 deleted the sedemche/fix_loading_indicator branch December 2, 2025 00:12
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.

4 participants