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

Editor and ScrollView problems in iOS #22946

Open
geertgeerits opened this issue Jun 10, 2024 · 8 comments · May be fixed by #26024
Open

Editor and ScrollView problems in iOS #22946

geertgeerits opened this issue Jun 10, 2024 · 8 comments · May be fixed by #26024
Assignees
Labels
area-controls-editor Editor area-controls-scrollview ScrollView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@geertgeerits
Copy link

Description

  1. Editor: HorizontalOptions does not work.
  2. ScrollView: Orientation="Vertical": The orientation is wrong when the device is rotated from landscape to portrait. The orientation is changed to horizontal.

Steps to Reproduce

Open the repository and click the 'Click me' button.

Link to public reproduction project repository

https://github.com/geertgeerits/MauiBugEditor

Version with bug

8.0.21 SR4.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS tested from version 15

Did you find any workaround?

No

Relevant log output

No response

@geertgeerits geertgeerits added the t/bug Something isn't working label Jun 10, 2024
Copy link
Contributor

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

Closed similar issues:

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

@Zhanglirong-Winnie Zhanglirong-Winnie added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jun 11, 2024
@Zhanglirong-Winnie
Copy link

Verified this issue with Visual Studio 17.11.0 Preview 1.1 (8.0.40&8.0.21). Can repro on iOS platform with sample project.

@jsuarezruiz jsuarezruiz added this to the Backlog milestone Jun 11, 2024
@samhouts samhouts removed s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 3, 2024
@samhouts samhouts added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 10, 2024
@karthikraja-arumugam
Copy link

Issue 1: Since the parent grid is set to center alignment, the editor is aligned according to its size. If you want the editor to occupy the entire width, set the grid's horizontal options to "Fill" so the editor can take up the available space.

    <Grid
        RowDefinitions="Auto, *, Auto"
        ColumnDefinitions="130*, 100, 130*, 1"
        HorizontalOptions="Fill"
        Margin="0,0,4,5">

       ...
</Grid>

Issue 2: There doesn't appear to be any issue when changing device orientation at runtime. Could you please provide additional details, such as a screenshot or video, for better clarity on the issue?

@geertgeerits
Copy link
Author

geertgeerits commented Sep 23, 2024

HorizontalOptions="Center" to HorizontalOptions="Fill" makes no difference. The error remains. Tested on an iPad with iOS 17.7.
See images on the project repository.
It works correct on Android and Windows.

@karthikraja-arumugam
Copy link

Yes @geertgeerits, I missed some code.
We need to set the horizontal option for the inner grid. The screenshot below shows the resulting output.
image (1)

If we set the horizontal options for both grids, the output will look like this.
image (2)

@geertgeerits
Copy link
Author

The first problem is solved with that but not the second. When you press the 'Click me' key in vertical position you see the full text. Turn the device to horizontal position and then back to vertical position then you notice that the text is not displayed completely. On the iPad the text is lost after OOOOOO-P. So the letters P, Q, R, S, T are not displayed.

@karthikraja-arumugam
Copy link

@geertgeerits, The issue with horizontal scrolling has been resolved in the latest .NET 9 package. Could you please verify this with .NET 9?

@karthikraja-arumugam karthikraja-arumugam added the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Oct 29, 2024
@geertgeerits
Copy link
Author

geertgeerits commented Oct 30, 2024

Hi,
The error is still there: ScrollView: Orientation="Vertical": The orientation is wrong when the device is rotated from landscape to portrait. The orientation is changed to horizontal.
When entering a text (click the 'Click me' button) in portrait position, then turn the device to landscape position and then back to portrait position, you wil see that there is text missing. Link to public reproduction project repository:
https://github.com/geertgeerits/MauiBugEditor
See screenshots on the repository: IMG_5471-Portrait before.PNG - IMG_5472-Landscape.PNG - IMG_5473-Portrait after.PNG
I think it has to do with the HorizontalOptions property of the editor.

Visual Studio 2022 Preview: version 17.12.0 Preview 5.0
Microsoft.Maui.Controls: version 9.0.0-rc.2.24503.2
Microsoft.Maui.Controls.Compatibility: version 9.0.0-rc.2.24503.2

@dotnet-policy-service dotnet-policy-service bot removed the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Oct 30, 2024
@NirmalKumarYuvaraj NirmalKumarYuvaraj linked a pull request Nov 21, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-editor Editor area-controls-scrollview ScrollView platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants