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

Tab key doesn't move between Entry fields on Mac Catalyst #8473

Closed
Tracked by #14021
davidortinau opened this issue Jul 1, 2022 · 6 comments
Closed
Tracked by #14021

Tab key doesn't move between Entry fields on Mac Catalyst #8473

davidortinau opened this issue Jul 1, 2022 · 6 comments
Labels
area-controls-entry Entry p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/macOS 🍏 macOS / Mac Catalyst s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version s/verified Verified / Reproducible Issue ready for Engineering Triage t/a11y Relates to accessibility t/bug Something isn't working

Comments

@davidortinau
Copy link
Contributor

Description

Form with 2 Entry fields and a Button. When I press the tab key, it adds tab spacing to the Entry with focus.

I expect it to move between fields by default. This is the behavior on Apple apps like Contacts. This is also the default behavior of SwiftUI.

//
//  ContentView.swift
//  Shared
//
//  Created by David Ortinau on 7/1/22.
//

import SwiftUI

struct ContentView: View {
    @State private var username: String = ""
    @State private var password: String = ""
    
    var body: some View {
        VStack{
            Text("Hello, world!")
                .padding()
            TextField("User name (email address)",
                      text: $username)
            TextField(
                    "Password",
                    text: $password
                )
                
        }
        
    }
}

struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}

Screen Shot 2022-07-01 at 12 39 35 PM

While I may be able to force this behavior, it seems it should be the default, and any other behavior should be additional work.

Screen Shot 2022-07-01 at 12 37 24 PM

Steps to Reproduce

Run this as a Mac Catalyst app, focus on the first Entry, then hit the Tab key to move between fields.
EntryTabbing.zip

Version with bug

6.0.400 (current)

Last version that worked well

Unknown/Other

Affected platforms

macOS

Affected platform versions

latest

Did you find any workaround?

No response

Relevant log output

No response

@davidortinau davidortinau added t/bug Something isn't working s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Jul 1, 2022
@Eilon Eilon added legacy-area-a11y Relates to accessibility legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels Jul 1, 2022
@jsuarezruiz jsuarezruiz added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Jul 4, 2022
@jsuarezruiz
Copy link
Contributor

Verified with the attached sample.

@agendamatic
Copy link

I've run into this as well. On Mac Catalyst, the tab \t literal is inserted into the text. While I can see use cases for inserting a tab into an Entry (or Editor), normal tab support should take precedence.

Per the documentation (at https://docs.microsoft.com/en-us/dotnet/maui/user-interface/controls/entry): "The Completed event is raised when the user has ended input by pressing the return key on the keyboard, or by pressing the Tab key on Windows." This would also apply to Mac Catalyst as a desktop environment with a keyboard.

I just tested this on Windows, and while the focus does move to the next focusable element, the Completed event is not invoked.

@PureWeen PureWeen added this to the .NET 8 Planning milestone Aug 3, 2022
@mikeparker104 mikeparker104 added the partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with label Jan 21, 2023
@Redth
Copy link
Member

Redth commented Jan 23, 2023

This appears to be working fine for me now in the latest service release. @davidortinau please close if you agree.

@Redth Redth added the s/try-latest-version Please try to reproduce the potential issue on the latest public version label Jan 23, 2023
@ghost
Copy link

ghost commented Jan 23, 2023

Hi @davidortinau. We have added the "s/try-latest-version" label to this issue, which indicates that we'd like you to try and reproduce this issue on the latest available public version. This can happen because we think that this issue was fixed in a version that has just been released, or the information provided by you indicates that you might be working with an older version.

You can install the latest version by installing the latest Visual Studio (Preview) with the .NET MAUI workload installed. If the issue still persists, please let us know with any additional details and ideally a reproduction project provided through a GitHub repository.

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@samhouts samhouts added the p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint label Jan 24, 2023
@tstephansen
Copy link

Not sure if this is related but the tab key also doesn't move between fields in a Maui Blazor app running on MacCatalyst.

@hartez
Copy link
Contributor

hartez commented Feb 1, 2023

I just tested this with the latest stable and the attached repro project, and it's working as expected.

@hartez hartez closed this as completed Feb 1, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Mar 3, 2023
@samhouts samhouts modified the milestones: .NET 8 Planning, .NET 8 Apr 6, 2023
@samhouts samhouts added the platform/macOS 🍏 macOS / Mac Catalyst label Feb 1, 2024
@Eilon Eilon added t/a11y Relates to accessibility and removed legacy-area-a11y Relates to accessibility legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor labels May 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-entry Entry p/1 Work that is important, and has been scheduled for release in this or an upcoming sprint partner/cat 😻 this is an issue that impacts one of our partners or a customer our advisory team is engaged with platform/macOS 🍏 macOS / Mac Catalyst s/triaged Issue has been reviewed s/try-latest-version Please try to reproduce the potential issue on the latest public version s/verified Verified / Reproducible Issue ready for Engineering Triage t/a11y Relates to accessibility t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

10 participants