Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

イベントログの設定画面のアクセシビリティを修正 #1068

Merged
1 commit merged into from
Jul 8, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jul 7, 2022

Issue 番号 / Issue ID

  • Close #

目的 / Purpose

  • イベントログの設定画面のアクセシビリティを修正

変更内容 / Changes

  • チェックボックスの読み上げを独自に行うようにした
    • 標準だとiOSはオン・オフを読み上げない
    • Androidは、CheckBoxにAutomationProperty.Nameを指定するとテキストも表示されてしまう
      • CheckBoxはTextプロパティは用意されておらず見た目も変えられない

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: A11y

確認事項 / What to check

  • [ ]

その他 / Other information


Internal IDs:

  • Product Backlog Item 7404

x:Class="Covid19Radar.Views.EventLogSettingPage"
ios:Page.UseSafeArea="true"
Style="{StaticResource DefaultPageStyle}"
Title="{x:Static resources:AppResources.EventLogSettingPageTitle}">
<ContentView>
<StackLayout>
<Label
AutomationProperties.IsInAccessibleTree="True"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

画面最初の要素でこれを指定(TrueでもFalseでも)すると、画面遷移時のTalkBackの挙動がおかしくなる。

@ghost ghost requested review from a user, cocoa-dev003 and cocoa-dev009 July 7, 2022 05:41
@ghost ghost self-assigned this Jul 7, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jul 7, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

8.3% 8.3% Coverage
0.0% 0.0% Duplication

@ghost ghost merged commit ae6e055 into develop Jul 8, 2022
@ghost ghost deleted the feature/event-log-a11y branch July 8, 2022 02:24
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant