Skip to content

fix(no-ie): add js snipped to check for internet explorer (#445) #519

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.

Sign up for GitHub

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

Open
VanLampe wants to merge 8 commits into
base: master
Choose a base branch
from
Open

fix(no-ie): add js snipped to check for internet explorer (#445) #519

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5dc9f7c
fix(no-ie): add js snipped to check for internet explorer (#445)
VanLampe Mar 10, 2020
9a25012
fix(no-ie): remove unnecessary script from index (#519)
VanLampe Mar 10, 2020
f12f03a
fix(no-ie): resolve comment
VanLampe Mar 10, 2020
6227bd7
fix(no-ie): remove AxtiveX elements
VanLampe Mar 11, 2020
c002afb
fix(no-ie): wrap ie-detection in one line
VanLampe Mar 11, 2020
4e65df1
fix(no-ie):resolve comment
VanLampe Mar 11, 2020
ce2136c
fix(no-ie): resolve comment
VanLampe Mar 11, 2020
2ff190e
fix(no-ie): add focus to <a> element
VanLampe Mar 11, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 27 additions & 1 deletion Phonebook.Frontend/src/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
<meta name="viewport" content="width=device-width, initial-scale=1" />
<link rel="icon" type="image/x-icon" href="favicon.ico" />
</head>

<body class="mat-typography">
<app-root></app-root>
<noscript>Please enable JavaScript to continue using this application.</noscript>
Expand All @@ -47,5 +46,32 @@
} else {
top.location = self.location;
}
msieversion();

function msieversion() {
var browserList = {
//list of browser-links where app can be openend in: key=browser name, value=link for browser download
'Google Chrome': 'https://www.google.de/chrome/',
'Mozilla Firefox': 'https://www.mozilla.org',
Opera: 'https://www.opera.com'
};
var ua = window.navigator.userAgent;
var msie = ua.indexOf('MSIE ');

if (msie > 0 || !!navigator.userAgent.match(/Trident.*rv\:11\./)) {
// stone age browser IE
var html =
'<div class="pb-no-ie"><h1>This application does not support Internet Explorer!</h1><p>To run the Phonebook, please <a href="microsoft-edge:' +
window.location.href +
'">open it in Microsoft Edge</a></strong> or use one of the following modern browsers:</p><ul><strong>';
for (var key in browserList) {
html += '<li><strong><a href="' + browserList[key] + '" target="_blank">' + key + '</a></strong></li>';
}
html +=
'</ul><p>If this was detected by mistake consider <a href="https://github.com/T-Systems-MMS/phonebook/issues/new/choose">opening an issue</a>.</p> </div>';

document.getElementsByTagName('body')[0].innerHTML = html;
}
}
</script>
</html>
13 changes: 12 additions & 1 deletion Phonebook.Frontend/src/styles.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,18 @@ app-root {
margin: auto 1%;
}
}

.pb-no-ie {
top: 38%;
left: 38%;
position: absolute;
Comment on lines +185 to +187
Copy link
Collaborator

@DanielHabenicht DanielHabenicht Mar 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
top: 38%;
left: 38%;
position: absolute;
margin: auto;
height: 200px;

Copy link
Collaborator

@DanielHabenicht DanielHabenicht Mar 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

damit ist es auch genau in der Mitte

}
.pb-no-ie a {
cursor: pointer;
Copy link
Collaborator

@DanielHabenicht DanielHabenicht Mar 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

das ist eigentlich überfüssig, <a> sind immer mit pointer belegt :)

text-decoration: none;
&:hover {
color: #9b9e9e;
}
}
//Media query classes
@include on-med-and-up {
.pb-hide-on-med-and-up {
Expand Down