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

Add firefox as supported browser #1186

Closed

Conversation

nielsvanvelzen
Copy link

I recently switched to Firefox as my primary browser and was surprised to see Tidepool blocking browsers without any clear reason. As explained in the mentioned issues, if there are specific functionalities that require Chrome you should use feature testing instead of blocking a complete browser. This is a really dark UX pattern.

This PR adds Firefox to the "supported browsers" list, which obviously is still a hack because Tidepool still blocks browsers for not being Chrome.

Fixes #418
Fixes #1029

@krystophv
Copy link
Member

Thanks for this, you can follow along with our efforts to expand browser support here: https://tidepool.atlassian.net/browse/WEB-1806

@nielsvanvelzen
Copy link
Author

Thanks for the fast response. It's great to see Firefox support is already in progress. The linked issues didn't have any official responses so I wasn't aware of that. It does look like the last changes to the ticket happened back in August, does that mean the project is currently on hold?

Would it be possible to create some kind of "ignore this warning" button in the meantime as an easy-win for the Firefox users?

@krystophv
Copy link
Member

The linked issue has many sub-tasks which describe testing and bug reports from cross-browser verification. Firefox support is the next priority and, to my knowledge, the testing and bug fixes for Firefox are now complete. All that remains is the task to adjust our supported browser list and page. I'll bring this up in the next sprint prioritization for consideration.

We cannot add a warning bypass for browsers that we haven't yet verified and documented functionality for.

@nielsvanvelzen
Copy link
Author

Thanks for the responses, appreciate the efforts! I'll close the PR so you guys can add the support properly.

@nielsvanvelzen nielsvanvelzen deleted the support-firefox branch March 3, 2023 18:51
@vitawasalreadytaken
Copy link

Hi, apologies if I'm hijacking an old thread, but this seems to be the one place on GitHub which aggregates several duplicate issues on the topic of browser compatibility.

For what it's worth, I'd like to express support for this bugfix as well. I deliberately use the word "bugfix" rather than "feature". Google Chrome is a browser that comes with serious privacy issues (just like its parent company, and that's no accident). I am disappointed that I am being forced to use it if I want to use Tidepool. I am a big fan of Tidepool's open-source ethos, and this odd privacy-harming restriction doesn't quite fit it 😕

I found some quite recent updates on child issues of https://tidepool.atlassian.net/browse/WEB-1806, so it seems there is some progress on this. Perhaps the delivery could be sped up by expanding browser support one by one? 🙏

(I am aware that I can circumvent this block by modifying the User-Agent string in the meantime.)

Thank you for your work – I hope this doesn't come across as ungrateful. It is amazing how much quality software you put out there for free use!

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

Successfully merging this pull request may close these issues.

Blip requires Chrome, although site works fine in Firefox Only works in Google Chrome
3 participants