search

dotnet / maui

.NET MAUI is the .NET Multi-platform App UI, a framework for building native device applications spanning mobile, tablet, and desktop.
https://dot.net/maui
MIT License
21.94k stars 1.7k forks source link

[Bug] SemanticHeading limitations #3117

Open rachelkang opened 2 years ago

rachelkang commented 2 years ago

Description

SemanticProperties.HeadingLevel doesn't work when applied to certain controls like SearchBar and ContentPage.ToolbarItem. While it doesn't work for SearchBar, the app crashes when applied to ToolbarItem.

Investigated only on Android

Steps to Reproduce

  1. Open .NET MAUI sln on recipes branch
  2. Set SemanticProperties.HeadingLevel on the SearchBar in StartingPage.xaml
  3. Set SemanticProperties.HeadingLevel on the ToolbarItem in MyRecipesPage.xaml
  4. Deploy app to Android physical device
  5. Turn on TalkBack
  6. Activate Headings mode if not already activated (use three finger horizontal swipe gears)
  7. Swipe up/down to navigate Headings on StartingPage - SearchBar will not be reached
  8. Navigate to MyRecipes page - app will crash

Expected Behavior

SemanticProperties.HeadingLevel should be applicable to ToolbarItem, as it functions with Button, a similar control. Whether it is applicable to other controls like SearchBar should be clarified and accurately reflected (in intellisense, for example).

Actual Behavior

SemanticProperties.HeadingLevel doesn't work for SearchBar and ToolbarItem.

Basic Information

v-longmin commented 2 years ago

@rachelkang Could you share your repro project, thanks?

ghost commented 2 years ago

Hi @rachelkang. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. 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.

ghost commented 1 year ago

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

QianaJiao commented 1 year ago

Verified this issue with Visual Studio Enterprise 17.6.0 Preview 2.0. Not repro on Android device with Maui Project. TabbedPage.zip

ghost commented 1 year ago

Hi @rachelkang. 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.

rachelkang commented 1 year ago

Hi, @QianaJiao - I've tested again on .NET 8 and can confirm this is still an issue. While it doesn't crash, heading is not supported on searchBar or toolBarItem.

The repro project referenced is at https://github.com/rachelkang/recipeSearch

That being said, please don't worry about re-validating this issue as I'm on the .NET MAUI team and have validated that it is. Thank you!

ramonB1996 commented 4 months ago

This is also an issue with ListViews and even CollectionViews: https://github.com/dotnet/maui/issues/21830