Fix Shell flyout template type cast errors on Android - #3119
Conversation
|
Learn Build status updates of commit 40f5c00: ✅ Validation status: passedFor more details, please refer to the build report. |
Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 312808b: ✅ Validation status: passed
For more details, please refer to the build report. |
|
@copilot VisualStateManager is on the Grid(FlyoutItemLayout) itself, specifying TargetName="FlyoutItemLayout" is redundant and you're targeting the same element that already owns the VSM. So, remove this changes. |
Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com>
Removed the redundant |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 19b9cda: ✅ Validation status: passed
For more details, please refer to the build report. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 2a81687: ✅ Validation status: passed
For more details, please refer to the build report. |
AppThemeBindingnested insideOnPlatformelement syntax which causes type cast errors on Android<OnPlatform x:TypeArguments="Color">withAppThemeBindingforTextColorTextColor="{OnPlatform Android={AppThemeBinding Light=#DE000000, Dark=White}}"Selectedvisual stateBackgroundColorto use correct alpha values (#1A000000and#1AFFFFFF) matching MAUI sourceTargetName="FlyoutItemLayout"from Selected state setter (VSM is already on the Grid)ms.dateto reflect the documentation changeOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Internal previews