Skip to content
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

[iOS] Spacing between Header and FlyoutItems on iOS only #11679

Closed
AndreasReitberger opened this issue Nov 27, 2022 · 6 comments · Fixed by #12480
Closed

[iOS] Spacing between Header and FlyoutItems on iOS only #11679

AndreasReitberger opened this issue Nov 27, 2022 · 6 comments · Fixed by #12480
Labels
area-controls-flyout Flyout area-controls-shell Shell Navigation, Routes, Tabs, Flyout fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! platform/iOS 🍎 t/bug Something isn't working
Milestone

Comments

@AndreasReitberger
Copy link

Description

When I'm running my app on iOS, I get a space between the HeaderTemplate and the first FlyoutItem.
Running the same project on Android is working fine.

<Shell.FlyoutHeader>
        <Grid
            HeightRequest="92"
            BackgroundColor="{AppThemeBinding Light={StaticResource Gray800}, Dark={StaticResource Gray200}}"
            >
            <Label
                Text="Just a simple header"
                TextColor="{AppThemeBinding Light={StaticResource Black}, Dark={StaticResource White}}"
                />
        </Grid>
    </Shell.FlyoutHeader>

    <FlyoutItem
        Title="MainpPage" 
        >
        <ShellContent
            ContentTemplate="{DataTemplate local:MainPage}"
            Route="DashboardPage"
            />
    </FlyoutItem>

iOS

2022-11-27_07-34-39-

Android

Screenshot_1669531325

Steps to Reproduce

  1. User the repro provided
  2. Run on Android and iOS
  3. Compare both flyout menus

Link to public reproduction project repository

https://github.com/AndreasReitberger/ReproScrollingIssueModal

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 16, haven't tried any older version

Did you find any workaround?

No response

Relevant log output

No response

@AndreasReitberger AndreasReitberger added the t/bug Something isn't working label Nov 27, 2022
@nebula2
Copy link

nebula2 commented Nov 27, 2022

why the heightrequest on the grid? how does it behave when you remove that?

@jsuarezruiz jsuarezruiz added platform/iOS 🍎 area-controls-shell Shell Navigation, Routes, Tabs, Flyout area-controls-flyout Flyout labels Nov 28, 2022
@jsuarezruiz jsuarezruiz added this to the Backlog milestone Nov 28, 2022
@ghost
Copy link

ghost commented Nov 28, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@AndreasReitberger
Copy link
Author

why the heightrequest on the grid? how does it behave when you remove that?

Removing the HeightRequest results in the same output.

2022-11-29_08-16-25-

@GnanaPriyaNamasivayam
Copy link

When can we expect this fix to live, Any idea?

@Greg-Bates-Trimble
Copy link

Just following up on this. Is there any update on when this fix will go live? Thanks

@GnanaPriyaNamasivayam
Copy link

Any update on when is the new ongoing release is expected to live.

@samhouts samhouts added the fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! label Apr 12, 2023
@ghost ghost locked as resolved and limited conversation to collaborators May 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-flyout Flyout area-controls-shell Shell Navigation, Routes, Tabs, Flyout fixed-in-8.0.0-preview.3.8149 Look for this fix in 8.0.0-preview.3.8149! platform/iOS 🍎 t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants