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.

Sign up for GitHub

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

Jump to bottom

[A11Y] Notification clear buttons have small touch targets #3518

Open
Tracked by #3360
davwheat opened this issue Aug 22, 2021 · 1 comment
Open
Tracked by #3360

[A11Y] Notification clear buttons have small touch targets #3518

davwheat opened this issue Aug 22, 2021 · 1 comment
Assignees
@davwheat
Labels
type/accessibility Issues relating to accessibility (keyboard navigation, screenreaders, text contrast, etc.)

Comments

@davwheat
Copy link
Member

davwheat commented Aug 22, 2021 •
edited
Loading

image
@jordanjay29
Copy link
Contributor

Ideally, that button would fill the available space for the NotificationList-header div above it, at least the full vertical space and the space to the right. Not sure how far to go on the left (perhaps just until it's square).

The individual notif clear buttons could use some attention as well.

image

@askvortsov1 askvortsov1 transferred this issue from flarum/framework Mar 10, 2022
@davwheat davwheat mentioned this issue Mar 28, 2022
Open
28 tasks
@davwheat davwheat self-assigned this Jul 11, 2022
@davwheat davwheat transferred this issue from flarum/issue-archive Jul 11, 2022
@davwheat davwheat added the type/accessibility Issues relating to accessibility (keyboard navigation, screenreaders, text contrast, etc.) label Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davwheat davwheat

Labels
type/accessibility Issues relating to accessibility (keyboard navigation, screenreaders, text contrast, etc.)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davwheat @jordanjay29