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

[a11y] 9.1.4.10 Break content #10728

Closed
1 of 3 tasks
tschiebel opened this issue Apr 5, 2024 · 9 comments · Fixed by #11344
Closed
1 of 3 tasks

[a11y] 9.1.4.10 Break content #10728

tschiebel opened this issue Apr 5, 2024 · 9 comments · Fixed by #11344
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Topic:Accessibility Type:Bug Something isn't working

Comments

@tschiebel
Copy link

tschiebel commented Apr 5, 2024

audit: https://infinite.owncloud.com/s/PnaAukHGGsLnrMM

https://bitvtest.de/pruefschritt/bitv-20-web/bitv-20-web-9-1-4-10-inhalte-brechen-um

Personal

  • With a browser window width of 1280 CSS pixels and a zoom magnification of
    400%, the page content does not break. This means that all information and
    functions are not available without users having to scroll horizontally.
    image

Shares

  • If the browser window width is set to 1280 CSS pixels and the zoom magnification is set to
    400%, only two columns are displayed in the table: "Name" and "Actions".
    "Actions". Other table columns are no longer accessible to users. To
    ensure that all table columns are visible even at higher zoom magnification, the table should either be
    the table should either be designed in such a way that it adapts to the available
    width or offer the option of scrolling horizontally through the table columns.
    scroll horizontally through the table columns.
    image

Account

  • If the browser window width is set to 1280 CSS pixels and the zoom magnification is set to
    400%, the content at the bottom of the page is cut off and is no longer accessible.
    no longer accessible.
    image
@pascalwengerter
Copy link
Contributor

Assigning @tbsbdr here since I need a decision on how to handle the tables on small screens

@pascalwengerter
Copy link
Contributor

Assigning myself to test the general "break content" issues once more.

Having a reduced functionality in the table is fine according to @tbsbdr since the underlying actions are still available through the (right) sidebar on mobile

@tbsbdr tbsbdr added this to the Web Accessibility milestone Jul 25, 2024
@tbsbdr tbsbdr added Priority:p2-high Escalation, on top of current planning, release blocker and removed Priority:p3-medium Normal priority labels Jul 26, 2024
@tbsbdr tbsbdr moved this from Qualification to Backlog in Infinite Scale Team Board Jul 26, 2024
@tbsbdr
Copy link
Contributor

tbsbdr commented Jul 29, 2024

  • remove text from buttons "New" and "Upload" for like 1280@400%

Image

  • Shared and Spaces should not scroll horizontally (lets make it like in personal, i.e. stack and remove)

Image

Image

@JammingBen JammingBen moved this from Backlog to In progress in Infinite Scale Team Board Aug 1, 2024
@kulmann
Copy link
Member

kulmann commented Aug 5, 2024

@tbsbdr at 400% zoom on a 14 inch macbook the file list is not visible at all 🙈 see screenshot:

Screenshot 2024-08-05 at 05 42 55

The table header is visible. Since the area above the table doesn't scroll I can't scroll the actual files into view. At 300% it still kind of works - I can see two rows of the file list (see next screenshot):

Screenshot 2024-08-05 at 05 46 13

Do we have to fix this or can we assume a different browser window height if someone uses 400% zoom? 🤔

@tbsbdr
Copy link
Contributor

tbsbdr commented Aug 5, 2024

uff, how about making the files top bar scrollable eg. if a certain zoom is reached? just thinking aloud..

@JammingBen
Copy link
Collaborator

Or just "unstick" the area above the table?

@tbsbdr
Copy link
Contributor

tbsbdr commented Aug 5, 2024

eh, yes, thats what I wanted to say :)

@kulmann
Copy link
Member

kulmann commented Aug 5, 2024

Yep, good idea. For normal desktop resolutions and 100% zoom level I like the sticky header. But for other situations that would be probably the easiest and most accessible solution.

@JammingBen JammingBen moved this from In progress to Done in Infinite Scale Team Board Aug 5, 2024
@JammingBen
Copy link
Collaborator

JammingBen commented Aug 8, 2024

Yep, good idea. For normal desktop resolutions and 100% zoom level I like the sticky header. But for other situations that would be probably the easiest and most accessible solution.

I'm just not sure if we should couple this to the zoom level. Some users have a default zoom setting of e.g. 110% or 125%. I feel like we still want to have the area sticky in this case.

Maybe couple it to the screen width? Because that's what basically happens with a zoom level of 400%: the screen width gets extremely small. So I suggest only doing this for screen widths <420px or <640px.

Edit: Or even better: couple this to the vertical window size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Topic:Accessibility Type:Bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants