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

Distinguish new notifcations for accessibility #4158

Closed
RickDBCN opened this issue Sep 23, 2022 · 1 comment · Fixed by #4159
Closed

Distinguish new notifcations for accessibility #4158

RickDBCN opened this issue Sep 23, 2022 · 1 comment · Fixed by #4159
Labels

Comments

@RickDBCN
Copy link
Contributor

Package

filament/notifications

Package Version

v2.2.16.11

Laravel Version

9.x

Livewire Version

No response

PHP Version

8.1.x

Problem description

Several screens can't see the difference between unread notifications and read notifications. See: https://prnt.sc/G6sXbDlmmwRn

Expected behavior

bg-gray-50 should be bg-primary-50 for increased accessibility. See change: https://prnt.sc/J-bu21Zn0xJe

Steps to reproduce

//

Reproduction repository

//

Relevant log output

No response

@RickDBCN RickDBCN added bug Something isn't working unconfirmed labels Sep 23, 2022
@github-actions
Copy link

Hey @RickDBCN! We're sorry to hear that you've hit this issue. 💛

However, it looks like you forgot to fill in the reproduction repository URL. Can you edit your original post and then we'll look at your issue?

We need a public Git repository which contains a Laravel app with the minimal amount of Filament code to reproduce the problem. That would allow us to download it and review your bug much easier, so it can be fixed quicker. Please make sure to include a database seeder with everything we need to set the app up quickly.

Also, it doesn't look like you've provided much information on how to replicate the issue. Please edit your original post with clear steps we need to take.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 23, 2022
@danharrin danharrin reopened this Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants