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.

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

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

Open
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 all 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
32 changes: 31 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,36 @@
} else {
top.location = self.location;
}
msieversion();

VanLampe marked this conversation as resolved.
Show resolved Hide resolved
</script>
<!-- Show Browser not supported Message -->
<script>
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 msie = window.navigator.userAgent.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:' +
mschwrdtnr marked this conversation as resolved.
Show resolved Hide resolved
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" rel="nofollow">' + 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

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

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

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, &:focus {
color: #9b9e9e;
}
}
//Media query classes
@include on-med-and-up {
.pb-hide-on-med-and-up {
Expand Down