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

Keyboard Scrolling in editors with Center or End VerticalTextAlignment is off #24977

Open
tj-devel709 opened this issue Sep 27, 2024 · 2 comments · May be fixed by #25827
Open

Keyboard Scrolling in editors with Center or End VerticalTextAlignment is off #24977

tj-devel709 opened this issue Sep 27, 2024 · 2 comments · May be fixed by #25827
Assignees
Labels
area-controls-editor Editor area-keyboard Keyboard, soft keyboard i/regression This issue described a confirmed regression on a currently supported version p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint platform/iOS 🍎 s/triaged Issue has been reviewed t/bug Something isn't working
Milestone

Comments

@tj-devel709
Copy link
Member

tj-devel709 commented Sep 27, 2024

Description

When you have a large Editor with the VerticalTextAlignment set to Center or End on iOS, both the keyboard scrolling inside KeyboardAutoManagerScroll and the MauiTextView.ShouldCenterVertically are working at the same time counterproductively to satisfy keeping the cursor above the keyboard and keeping the text aligned to the center or bottom. You'll see once the, editor's content is greater than the size of the editor, the MauiTextView.ShouldCenterVertically method will stop having effect and then just the KeyboardAutoManagerScroll will be keeping the cursor above the keyboard.

KeyboardScrollingEditorAlignment.mov

Steps to Reproduce

Video is from UITest - Issue19214_2

 <Grid RowDefinitions="50, 50, *, 50" Margin="30">
        <Entry Text="Content before" AutomationId="EntryBefore" FontSize="Large" ReturnType="Next" BackgroundColor="Aquamarine" Grid.Row="0" />
        <Label x:Name="CursorHeightTracker" Text="0" AutomationId="CursorHeightTracker" FontSize="Large" BackgroundColor="Aquamarine" Grid.Row="1" />
        <Editor x:Name="editor" AutomationId="IssueEditor" FontSize="Large" BackgroundColor="Orange" Grid.Row="2" VerticalTextAlignment="Center" TextChanged="Editor_TextChanged" />
        <Button Text="Erase" Grid.Row="3"/>
    </Grid>

Link to public reproduction project repository

No response

Version with bug

8.0.90 SR9

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 17

Did you find any workaround?

The keyboard scrolling can be disabled or the editor can be moved to a location above the keyboard

Relevant log output

No response

@tj-devel709 tj-devel709 added t/bug Something isn't working platform/iOS 🍎 p/2 Work that is important, but is currently not scheduled for release area-keyboard Keyboard, soft keyboard area-controls-editor Editor labels Sep 27, 2024
@tj-devel709 tj-devel709 added this to the Backlog milestone Sep 27, 2024
@tj-devel709 tj-devel709 self-assigned this Sep 27, 2024
Copy link

We've found some similar issues:

If any of the above are duplicates, please consider closing this issue out and adding additional context in the original issue.

Note: You can give me feedback by 👍 or 👎 this comment.

@PureWeen PureWeen added the i/regression This issue described a confirmed regression on a currently supported version label Oct 9, 2024
@PureWeen PureWeen modified the milestones: Backlog, .NET 9.0 GA Oct 9, 2024
@PureWeen PureWeen added p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint and removed p/2 Work that is important, but is currently not scheduled for release labels Oct 9, 2024
@PureWeen PureWeen modified the milestones: .NET 9.0 GA, .NET 9 SR1 Oct 14, 2024
@tj-devel709
Copy link
Member Author

tj-devel709 commented Oct 16, 2024

Also note that jumping in the editor when the text is clicked is undesirable behavior which starts with this PR: #24589

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-editor Editor area-keyboard Keyboard, soft keyboard i/regression This issue described a confirmed regression on a currently supported version p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint platform/iOS 🍎 s/triaged Issue has been reviewed t/bug Something isn't working
Projects
Status: Cut from this Milestone
4 participants