[macOS Avatar] Fix custom border drawing when built with Xcode 15 #1921
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Platforms Impacted
Description of changes
In AvatarView's drawRect override, we're drawing relative to the
dirtyRect
parameter instead of drawing to our bounds. This was already technically a bug becausedirtyRect
since this could have been incorrect ifsetNeedsDisplayInRect:
was used to invalidate part of the view. In practice that probably wasn't happening. But nowNSView.clipsToBounds
defaults to true when building with Xcode 15+ and running on macOS 14.0+ so AppKit now often gives us adirtyRect
which extends beyond the bounds of our view. That makes this much more obviously incorrect.To fix, just draw the border relative to our bounds.
Binary change
No impact
Verification
See screenshots
Visual Verification
|
Pull request checklist
This PR has considered:
Microsoft Reviewers: Open in CodeFlow