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

[BITV] 9.1.4.10/6.2 - At a browser window width of 320 CSS pixels (or at a browser window width of 1280 CSS pixels and 400% zoom magnification), the page contents do not wrap - some information and functions are not available without users having to scroll horizontally. (1) #36965

Closed
AndyScherzinger opened this issue Mar 1, 2023 · 8 comments
Assignees
Labels
Milestone

Comments

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Mar 1, 2023

The issue affects both Firefox and Chromium-based browsers, with the whole page being affected in the case of Chromium-based browsers.

Page — Personal info
Personal info - Personal settings - Nextcloud
https://try.nextcloud.com/ltd/a11y/index.php/settings/user

Details

https://report.bitvtest.de/default-en/d63601ac-cb34-4645-8256-66bec78964a0.html#checkpoint-81d891d46b-v6-n2

@AndyScherzinger AndyScherzinger added 1. to develop Accepted and waiting to be taken care of accessibility labels Mar 1, 2023
@AndyScherzinger AndyScherzinger changed the title [BITV] 9.1.4.10/6.2 - At a browser window width of 320 CSS pixels (or at a browser window width of 1280 CSS pixels and 400% zoom magnification), the page contents do not wrap - some information and functions are not available without users having to scroll horizontally. The issue affects both Firefox and Chromium-based browsers, with the whole page being affected in the case of Chromium-based browsers. (1) [BITV] 9.1.4.10/6.2 - At a browser window width of 320 CSS pixels (or at a browser window width of 1280 CSS pixels and 400% zoom magnification), the page contents do not wrap - some information and functions are not available without users having to scroll horizontally. (1) Apr 21, 2023
@JuliaKirschenheuter JuliaKirschenheuter self-assigned this Oct 6, 2023
@JuliaKirschenheuter
Copy link
Contributor

JuliaKirschenheuter commented Oct 10, 2023

General @JuliaKirschenheuter

Status message dialog @JuliaKirschenheuter

Search contacts @JuliaKirschenheuter

Search @JuliaKirschenheuter

Files

Files @JuliaKirschenheuter

Files left sidebar @ShGKme

  • [BITV] Fix left sidebar from a11y perspective #40954

  • 320px: sidebar button (navigation toggle) is not reachable
    Screenshot from 2023-10-10 14-15-42

  • 1280px + 400% zoom: same

  • 320px: vertical scrollbar is visible twice (should be visible once)
    Screenshot from 2023-10-10 14-23-39

  • 1280px + 400% zoom: same

  • 320px: if sidebar is open it is possible to navigate through all other elements. This have to be prohibited.

Aufzeichnung.2023-10-11.145807.mp4

#36964 (comment)

  • 1280px + 400% zoom: same

Files right sidebar @JuliaKirschenheuter

  • Reduce width of sharing entry #40963

  • 320px: Sharing tab: vertical scrollbar is visible
    Screenshot from 2023-10-11 14-27-11

  • 1280px + 400% zoom: same

  • 320px: It is possible to navigate through all other elements on a page even if they are not visible. This have to be prohibited.

Peek 2023-10-11 15-30

  • 1280px + 400% zoom: same

Photos @JuliaKirschenheuter

Photos @JuliaKirschenheuter

New album modal window @JuliaKirschenheuter

Select the default location for your media modal window @JuliaKirschenheuter

Screenshot from 2023-10-10 16-39-59

Activity @susnux

Activity sidebar

Users @JuliaKirschenheuter

Users table

Apps @szaimen

Your apps

  • 320px: Sticky header makes hard to reach important buttons
    Screenshot from 2023-10-11 11-02-04

  • 1280px + 400% zoom: same

Personal settings @szaimen

Personal info

  • 320px: several elements are too large
    Screenshot from 2023-10-11 11-20-29
    solution:
    Labels + select should have flex-direction: column;

  • fix profile visibility settings being too wide #40932

  • 1280px + 400% zoom: same

  • 1280px + 400% zoom: Choose your profile picture dialog: Recent && Favorites have unneeded horizontal scrolling

Screenshot from 2023-10-11 11-23-31
solution:
don't load any table if content of a table is empty

Security

  • 320px: Dvices & Sessions labels + dropdowns are too large
    Screenshot from 2023-10-11 11-34-46
    solution:
    Labels + inputs should have flex-direction: column;

  • 1280px + 400% zoom: same

Sharing

  • 320px: several elements are too large
    Screenshot from 2023-10-11 11-52-12
    solution:
    flex-direction: column;

  • 1280px + 400% zoom: same

Availability

  • 320px: several elements are too large
    Screenshot from 2023-10-11 11-56-02
    solution: create right labels, datetime picker.
    flex-direction: column;

  • 1280px + 400% zoom: same

Administration settings @szaimen

Overview

  • 320px: info icon takes too much place
    Screenshot from 2023-10-11 13-35-44
    solution: place info icon somewhere else

  • 1280px + 400% zoom: same

Basic settings

  • 320px: label of inputs are broken + padding on section is too large
    Screenshot from 2023-10-11 13-39-17
    solution: design labels differently

  • 1280px + 400% zoom: same

Sharing

  • 320px: input field selectLinksExcludedGroups is too large
    Screenshot from 2023-10-11 13-46-15

  • 1280px + 400% zoom: same

@ShGKme
Copy link
Contributor

ShGKme commented Oct 11, 2023

Note:

In the alternative test with 1280px viewport width and 400% zoom, it should not be rated negatively if permanently positioned content (such as drop-out menus) is not fully accessible due to the very low available height of the browser window. The only check is whether content successfully breaks into a view that does not need to be scrolled horizontally. To check the loss of information and functionality, the test at 320px viewport width is crucial.

So only the width is checked, not the height.

@JuliaKirschenheuter
Copy link
Contributor

closing ticket. As follow up (not exactly related to this ticket with horizontal scrolling):

@ShGKme
Copy link
Contributor

ShGKme commented Nov 15, 2023

So only the width is checked, not the height.

UPD: this is not correct.

According to the requirement:

2.2 Alternative test with 1280px viewport width

  1. In a browser that supports zoom enlargement by 400% (e.g. B. Chrome) set the browser window to a width of 1280 CSS pixels and a height of 1024 CSS pixels. For this, the Resize-function of Web Developer Toolbar use.

Height 1024px + 400% or 256px also should work.

@ShGKme
Copy link
Contributor

ShGKme commented Nov 15, 2023

Files Sidebar Settings dialog Apps sidebar
image image settings apps

Most problematic places are sidebars and dialogs

@ShGKme
Copy link
Contributor

ShGKme commented Nov 15, 2023

To fix the sidebar, we can add:

.app-sidebar-tabs__content] {
  min-height: 512px; // or whatever, but not 0 px
}

@ShGKme
Copy link
Contributor

ShGKme commented Nov 15, 2023

The same with NcAppSettingsDialog

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

No branches or pull requests

4 participants