phonebook
phonebook copied to clipboard
T-Systems-MMS
fix(no-ie): add js snipped to check for internet explorer (#445)
resolves #445 check if internet explorer is used and offer links to more modern/common browsers
Sorry that it took so long, but I did not found time to start working at this. Please just tell me, what you guys think of the solution. I did not spend much time in css styling.
Bis auf die zwei Kommentare sieht es sehr gut aus. Kein Problem das es länger gedauert hat.
@DanielHabenicht thanks for your review! Does the calls to open the application with chrome, firefox and opera work for you? If I try that in the demo-environment it does not.
Nope but, my working on my comment should fix it.
I had to change security settings and enable ActiveX..now IE asks if I want to execute firefox for example.
I could not find a way to open the app in firefox, chrome or opera without ActiveX. Edge should work finde because I can call it via href..
I think we shouldn't use ActiveX in this context. Just provide a link where users can download the other browser. They can then switch it manually.
I think we shouldn't use ActiveX in this context. Just provide a link where users can download the other browser. They can then switch it manually.
I also think, ActiveX isnt a good solution. I will revoke those changes. But I will keep the Link to open with edge, because this works without ActiveX.
Wie machen wir das eigentlich mit der Übersetzung? Kannst du dafür eine Issue aufmachen?
Wie machen wir das eigentlich mit der Übersetzung? Kannst du dafür eine Issue aufmachen?
Meinst du denn, dass man dafür wirklich eine Übersetzung brauchst? Ich hatte das auch erst überlegt, aber fand dann den Aufwand etwas zu hoch, da die Message ja eigentlich recht klar ist..
Wie machen wir das eigentlich mit der Übersetzung? Kannst du dafür eine Issue aufmachen?
Meinst du denn, dass man dafür wirklich eine Übersetzung brauchst? Ich hatte das auch erst überlegt, aber fand dann den Aufwand etwas zu hoch, da die Message ja eigentlich recht klar ist..
dann vielleicht nur auf englisch?
Kannst du außerdem die Komponenten löschen die dadurch nun überflüssig geworden sind? IEWarningDialog etc.
Hey @FrankLambrette how is it going? If I am right you only need to provide an English text and don't need to translate it :)
Stuff to resolve this PR🎉 🎉
- [ ] Remove IEWarningDialog (Angular Dialog not index.html)
- [ ] Resolve Style Issues (Comments)
- [ ] Provide only English Dialog Text