corona-warn-app / cwa-wishlist

Central repository to collect community feature requests and improvements. The CWA development ends on May 31, 2023. You still can warn other users until April 30, 2023. More information:
https://coronawarn.app/en/faq/#ramp_down
Apache License 2.0
106 stars 14 forks source link

App is artificially locked to portrait mode although this is an accessibility issue #848

Closed Ein-Tim closed 1 year ago

Ein-Tim commented 1 year ago

Avoid duplicates

Technical details

Describe the bug

The CWA Android app is artificially locked to only display the UI in portrait mode, although the app supports landscape mode, as you can easily confirm by using two apps side by side in landscape mode. Here is a screenshot: Screenshot_20220716_143150_de rki coronawarnapp

Steps to reproduce the issue

  1. Open CWA
  2. Rotate the device to landscape orientation
  3. Notice that the app does not respond to the rotation change

Expected behaviour

The UI of the app needs to be available in landscape mode, as only having the UI in portrait mode is an accessibility issue, as you yourself identified in the Accessibility Statement by the Robert Koch Institute and the Federal Ministry of Health, where it says:

The content of the app is limited to portrait display, in contradiction to requirement 1.3.4 WCAG 2.1 or 11.1.3.4 EN 301 549 V3.2.1.

Possible Fix

Remove the lock which hinders the app to render the UI in landscape mode.

MikeMcC399 commented 1 year ago

@Ein-Tim

image

If you want to try this out yourself refer to https://developer.android.com/guide/topics/manifest/activity-element in the section android:screenOrientation.

The settings to allow or disallow landscape mode are in the AndroidManifest.xml file.

Ein-Tim commented 1 year ago

@MikeMcC399

Allowing landscape mode will exposure some formatting issues which are not noticeable in portrait mode e.g. the cut off head in the following:

That is true, however, in my opinion some cut off illustrations, etc. should a) not stop the team from updating the app to support landscape orientation and ship bug fixes for this in later updates b) be easily fixable

Other functions, like the display of a QR code for scanning do not work in landscape mode because the app does not display the complete QR code in landscape orientation. It is either cut off at the top or the bottom.

As you can tap on the QR code & then a full screen view of the QR code is opened, this is not a problem neither. Yes, the QR code should be displayed completely instantly, but with the tap on the QR code workaround it is for an initial release with landscape orientation support OK.

MikeMcC399 commented 1 year ago

@Ein-Tim

I would view support of landscape mode as a major enhancement for the app. It's very easy to allow the app to rotate, however to allow it to display information in a useful way in landscape mode would require development of new layouts. I wouldn't see that as a simple bug fixing exercise. It that were the case, then probably the developers would have enabled landscape mode a long time ago. Perhaps anyway they will be able to provide some feedback.

I didn't make any attempt to go through all functions of the app when I mentioned a couple of examples where the layout has issues in landscape mode.

https://developer.android.com/guide/topics/large-screens/support-different-screen-sizes gives some insight into the problems involved.

Ein-Tim commented 1 year ago

@MikeMcC399 I think there are two ways of integrating landscape mode. Either, the team just allows the app to rotate and fixes the issue where illustrations or text is cut off (this would be the easy way), but for sure the app could be optimized so that the form factor landscape is used in the design and new layouts are created (more workload).

Thanks for giving your feedback on this issue! I was just surprised that the app actually works in landscape if it is used in split scree, so I opened this issue.

fynngodau commented 1 year ago

CCTG has lifted the orientation lock to enable landscape mode.

Besides that the layout is obviously not optimized for landscape, no other problems have been found so far.

larswmh commented 1 year ago

Thanks for your report @Ein-Tim. We have created an internal ticket for it and will raise this topic internally. Internal Tracking ID: EXPOSUREAPP-13641


Corona-Warn-App Open Source Team

Ein-Tim commented 1 year ago

@mtwalli You have relabeled this issue from https://github.com/corona-warn-app/cwa-wishlist/labels/bug to https://github.com/corona-warn-app/cwa-wishlist/labels/feature%20request and transferred it to the wishlist without further comment why this was done.

I opened this issue as a bug because it seems like an accessibility issue could be fixed by just removing one flag (leaving out re-designing something for landscape).

What exactly is the plan here? The ticket is internally set to "Story"? In this case I would close this issue and open a new wishlist issue, directly asking for landscape support.

mtwalli commented 1 year ago

@Ein-Tim Yes, it is a story that should be implemented for both Android and iOS. iOS is locked too, right? From dev side, I think we should pay attention to some points before allowing that: 1- Whether the current screen design fits landscape orientation 2- Preserving state between different orientations. Android recreates activities in this case and we need to make sure that user experience is not disturbed by that.

You can of course close it or edit as you wish ;)

Ein-Tim commented 1 year ago

iOS is locked too, right?

Yep.

Thanks for your explanations, as said, I will close this issue now and open a new, dedicated one, for this.