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

Show alert/popup/warning notification when opening Care on older/unsupported browsers #8403

Closed
rithviknishad opened this issue Aug 22, 2024 · 4 comments · Fixed by #8417
Closed
Assignees
Labels
good first issue Good for newcomers

Comments

@rithviknishad
Copy link
Member

rithviknishad commented Aug 22, 2024

Is your feature request related to a problem? Please describe.
CARE may use functions that may not be available on older browsers of users. This may lead to certain pages within CARE to crash when such browser unsupported functions are invoked.

Describe the solution you'd like
Based on the browserslist configured for the project, show a alert/popup/warning notification when users open CARE on such older/unsupported browsers.

We could use the following to achieve it:

Additional context
Recent issue that was caused due to older browser unsupported:

@rithviknishad rithviknishad added the good first issue Good for newcomers label Aug 22, 2024
@github-project-automation github-project-automation bot moved this to Triage in Care Aug 22, 2024
@AdityaJ2305
Copy link
Contributor

Hey @rithviknishad, can you assign this issue to me?

@rithviknishad rithviknishad moved this from Triage to Up Next in Care Aug 23, 2024
@AdityaJ2305
Copy link
Contributor

AdityaJ2305 commented Aug 24, 2024

Screenshot 2024-08-24 at 10 02 31 PM Is the CSS for the `BrowserWarning` component okay, or should I adjust it? and Can I put this browserWarning comp in the ErrorPages folder in the component ?

@Adiflickzzz
Copy link

Screenshot 2024-08-24 at 10 02 31 PM

Is the CSS for the BrowserWarning component okay, or should I adjust it?

and Can I put this browserWarning comp in the ErrorPages folder in the component ?

In which browser did you face this issue can u plz say me ..

@AdityaJ2305
Copy link
Contributor

Screenshot 2024-08-24 at 10 02 31 PM Is the CSS for the `BrowserWarning` component okay, or should I adjust it? and Can I put this browserWarning comp in the ErrorPages folder in the component ?

In which browser did you face this issue can u plz say me ..

You can change the default browser for testing in the inspect so it will behave like that only . I had changed to opera mini which is unsupported as per browserlist in package.json

@rithviknishad rithviknishad moved this from Up Next to Review required in Care Sep 9, 2024
@github-project-automation github-project-automation bot moved this from Review required to Done in Care Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants