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

Ignore background images in OnboardingBackgroundView for VoiceOver users [Bugfix] #658

Merged
merged 4 commits into from
Mar 2, 2023

Conversation

sempruijs
Copy link
Contributor

@sempruijs sempruijs commented Mar 1, 2023

Pull Request Checklist

UI changes have been tested with:

  • iPhone and iPad simulators in portrait and landscape orientations.
  • Dark mode enabled and disabled.
  • Various sizes of dynamic type.
  • Voiceover enabled.

In the welcome screen when you boot the app for the first time, there are moving images that are not useful for VoiceOver users. This pr ignores them.

I do not know the flow of adding lines to the change log, so if someone could explain how that works, I would be glad :)
(It's my first time here)

@sempruijs
Copy link
Contributor Author

Signed-off-by: Sem Pruijs [email protected]

@pixlwave
Copy link
Member

pixlwave commented Mar 2, 2023

Thanks for your contribution @sempruijs, this PR looks good to me. If you could update your changelog filename to be pr-658.bugfix, I'll get this merged.

@sonarcloud
Copy link

sonarcloud bot commented Mar 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@sempruijs
Copy link
Contributor Author

Yesss did it 🙌

Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

Thanks very much @sempruijs, I'm sure VoiceOver users will appreciate this contribution 🙌

@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (9ace5ac) 52.31% compared to head (09f7b4e) 52.31%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #658      +/-   ##
===========================================
- Coverage    52.31%   52.31%   -0.01%     
===========================================
  Files          259      259              
  Lines        14187    14188       +1     
  Branches      9118     9118              
===========================================
  Hits          7422     7422              
- Misses        6543     6544       +1     
  Partials       222      222              
Flag Coverage Δ
unittests 20.82% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...reens/OnboardingScreen/View/OnboardingScreen.swift 43.24% <0.00%> (-0.30%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pixlwave pixlwave merged commit 9e604c1 into element-hq:develop Mar 2, 2023
@pannie
Copy link

pannie commented Mar 18, 2023

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants