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

Fixed saveDialog for firefox. #142

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThomasADavis
Copy link

I've tested this in firefox 40.0 under Linux, and it fixes the dialog problem I reported.

@FedeG
Copy link

FedeG commented Oct 2, 2015

@bugvish, @jheising, @kcampbell or anyone from the freeboard team can review this issue please?

@Hywan
Copy link

Hywan commented Nov 18, 2015

I have a similar PR here. This is not only for Firefox but for all browsers. The event variable is not defined without this.

@Hywan
Copy link

Hywan commented Nov 18, 2015

@ThomasADavis Please, can you rename your PR title to remove “for firefox”?

@markuman
Copy link

works for me

@genez
Copy link

genez commented Oct 16, 2016

Hello everybody, the fix works on Firefox and Safari.
It's a 15 characters change in a single line. Not too hard to accept as a pull request in one (1) year.

@Stephan35
Copy link

Hello,
Dialog "Save FREEBOARD" does Nothing on Firefox 49.0.2
Works fine on Microsoft Internet Explorer.
Thanks

chrisv2 added a commit to chrisv2/freeboard that referenced this pull request Nov 7, 2018
joed74 added a commit to joed74/freeboard-mqtt that referenced this pull request Aug 20, 2022
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.

6 participants