dotnet / dotnet/maui

Re-enable Issue2818 test on iOS/Catalyst - FlyoutPage RTL hamburger icon and flyout items not displaying

Open
#33,499 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area-controls-flyoutpage area-layout platform/ios platform/macos s/triaged t/bug
Dominant language
C#
Stars
23.3k
Forks
2k
Avg merge
1d 14h
Merged PRs (30d)
296

Description

The test for issue #2818 is currently completely disabled on iOS, Catalyst, and Windows platforms. The issue is related to Right-to-Left (RTL) FlyoutPage behavior where the hamburger icon and flyout items don't display correctly.

Current State

The entire test file is wrapped in a conditional compilation directive:

#if TEST_FAILS_ON_WINDOWS && TEST_FAILS_ON_CATALYST && TEST_FAILS_ON_IOS
// ... entire test ...
#endif

Test File

  • src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/XFIssue/Issue2818.cs

Related Issues

  • iOS FlyoutPage RTL issue: #26726

Test Details

Issue: Right-to-Left FlyoutPage in Xamarin.Forms Hamburger icon issue

Test scenario:

  1. With flow direction set to RightToLeft on iOS
  2. In landscape orientation
  3. Flyout items should be displayed correctly
  4. Root view should move and content should be visible

Test steps:

  1. Tap "OpenRootView"
  2. Wait for "CloseRootView"
  3. Tap "CloseRootView"
  4. Set orientation to landscape
  5. Tap "OpenRootView" again
  6. Verify "CloseRootView" position
  7. Verify content visibility

Expected Behavior

When using RTL flow direction on iOS in landscape orientation:

  • Flyout items should be displayed correctly
  • Root view should animate properly
  • Content should be visible

Acceptance Criteria

  • Investigate the root cause of RTL FlyoutPage issue on iOS/Catalyst
  • Fix the underlying platform issue (if not already fixed)
  • Remove iOS/Catalyst from the conditional compilation
  • Verify test passes on iOS and Catalyst platforms

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/XFIssue/Issue2818.cs and inspect why the test is conditionally disabled on iOS, Catalyst, and Windows. Run the listed RTL FlyoutPage scenario on iOS and Catalyst, then investigate related issue #26726 for context. Done means the platform issue is fixed, the iOS/Catalyst conditions are removed, and the test passes on both platforms.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp, ios
Domain
mobile-dev, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.