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

Minor UI improvements in HTML layout, alignment, security fixes as per Chrome's Lighthouse recommendations #53

Closed
wants to merge 8 commits into from
Prev Previous commit
Next Next commit
Fixed know security vulnerabilities in old package versions by refere…
…ncing latest versions
  • Loading branch information
vkirienko committed Jul 20, 2020
commit 3bb00758d3a19b346c48c77b8c33a238608cc481
15 changes: 8 additions & 7 deletions html/index.html
Original file line number Diff line number Diff line change
@@ -1,16 +1,17 @@
<!DOCTYPE HTML>
<html>

<head>
<meta charset="utf-8"/>
<head>
<meta charset="utf-8" />
<meta name="description" content="Improved dump1090-fa web interface">
Copy link
Contributor

@liamengland1 liamengland1 Jul 20, 2020

Choose a reason for hiding this comment

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

I don't think this is a good description, just my opinion.

How about: Tar1090 - Live flight tracking map brought to you by ADS-B Exchange and our worldwide network of feeders? Or something a little more SEO-ey? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree it sounds much better but just copied it from this repo readme. I think it is up to wiedehopf to change it.

<meta name="mobile-web-app-capable" content="yes">
<meta name="viewport" content="user-scalable=no, initial-scale=1.0, minimum-scale=1.0, maximum-scale=1.0, minimal-ui" />

<script src="jquery/jquery-3.4.1.min.js"></script>
<script src="jquery/jquery-3.5.1.min.js"></script>
<script src="early.js"></script>
<script src="jquery/jquery-ui-1.11.4.min.js"></script>
<script src="jquery/jquery-ui-1.12.1.min.js"></script>
<script src="jquery/jquery.ui.touch-punch.min.js"></script>
<link rel="stylesheet" href="jquery/jquery-ui-1.11.4-smoothness.css" />
<link rel="stylesheet" href="jquery/jquery-ui-1.12.1-smoothness.css" />
<link rel="stylesheet" href="ol/v630/ol.css" type="text/css" />
<link rel="stylesheet" type="text/css" href="style.css" />
<link rel="stylesheet" type="text/css" href="colors.css" />
Expand All @@ -20,7 +21,7 @@
<script src="defaults.js"></script>
<script src="config.js"></script>

<link rel="stylesheet" href="ol/ol-layerswitcher350.css" type="text/css"/>
<link rel="stylesheet" href="ol/ol-layerswitcher350.css" type="text/css" />
<script src="ol/ol-layerswitcher350.js"></script>

<script src="markers.js"></script>
Expand All @@ -34,7 +35,7 @@
<script src="script.js"></script>
<link rel="icon" type="image/png" href="images/tar1090-favicon.png">
<title>tar1090</title>
</head>
</head>

<body onload="initialize()">
<div id="loader" >
Expand Down
2 changes: 0 additions & 2 deletions html/jquery/jquery-3.4.1.min.js

This file was deleted.

2 changes: 2 additions & 0 deletions html/jquery/jquery-3.5.1.min.js

Large diffs are not rendered by default.

Loading