Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Bug 648002. Redesign nightly.mozilla.org. phase 1. r=sayrer

commit e6910f3e9a2963f3fb19535e285b8e5084bd0291 1 parent 199464b
@tofumatt tofumatt authored
View
BIN  blueGradient.png
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
View
BIN  firefox.png
Deleted file not rendered
View
36 nightly.css
@@ -1,34 +1,40 @@
h1 {
font-size: 360%;
- color: #fff;
- margin: 20px 0 0 0;
+ color: #333;
+ margin: 0;
}
li {
margin-bottom: 1em;
}
-html {
- background: url(http://www.mozilla.com/img/tignish/template/background-tile.png) repeat-x 50% -145px #fff;
-}
-
body {
+ font-family: Georgia, serif;
padding: 20px;
- background: url(http://www.mozilla.com/img/tignish/template/background-feature.jpg) 50% -145px no-repeat;
+ background: url(blueGradient.png) top left repeat-x;
+ color: #333333;
@cvan
cvan added a note

you used #333 above. for consistency's and brevity's sakes, this should also be #333.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
}
-ul {
- width: 500px !important;
+#builds {
+ margin-left: auto;
+ margin-right: auto;
+ text-align: left;
+ width: 800px;
}
ul,p,#main-feature {
- width: 900px;
+ width: 800px;
text-align: left;
margin-left: auto;
margin-right: auto;
left: 0;
}
+ul,p {
+ margin-left: 125px;
+ width: auto;
+}
+
p {
clear: left;
padding-top: 50px;
@@ -42,8 +48,8 @@ ul li {
}
ul li a {
- font-weight: bold;
- font-family: verdana, sans-serif;
+ font-weight: bold;
+ font-family: Georgia, serif;
background-position: 0 50%;
background-repeat: no-repeat;
display: block;
@@ -53,7 +59,7 @@ ul li a {
ul li small {display: block;
font-size: 65%;
- font-family: verdana, sans-serif;
+ font-family: Georgia, serif;
margin-top: 3px;
}
@@ -70,8 +76,8 @@ ul li.linux a {
}
#main-feature {
- padding: 0;
- background: url(firefox.png) 0 0 no-repeat;
+ padding: 5px 0 0 5px;
+ background: url(nightly.png) 0 0 no-repeat;
@cvan
cvan added a note

0 0 is unnecessary

@tofumatt Owner

This is three years old and already committed. It's probably outdated. File a pull request if you have issues, please don't code review ancient code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
}
#main-feature p {
View
BIN  nightly.png
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
View
30 scrape.py
@@ -34,7 +34,6 @@ def __init__(self, suffix, extension, name, css_class):
BuildDisplay(".win32.installer", "exe", "Windows", "windows exe"),
BuildDisplay(".win64-x86_64.installer", "exe", "Windows 64-bit", "windows exe x64"),
BuildDisplay(".mac", "dmg", "Mac", "mac dmg"),
- BuildDisplay(".mac64", "dmg", "Mac 32/64-bit Universal", "mac dmg x64"),
BuildDisplay(".linux-i686", "tar.bz2", "Linux Intel", "linux bz2"),
BuildDisplay(".linux-x86_64", "tar.bz2", "Linux 64-bit Intel", "linux bz2 x64")
]
@@ -94,7 +93,6 @@ def buildHTML(recent_builds, parse_url, title):
<html>
<head>
<title>Firefox Nightly Builds</title>
- <link rel="stylesheet" type="text/css" href="http://www.mozilla.com/style/tignish/template.css" />
<link rel="stylesheet" type="text/css" href="http://www.mozilla.com/style/tignish/content.css" />
<link rel="stylesheet" type="text/css" href="nightly.css" />
</head>
@@ -103,23 +101,29 @@ def buildHTML(recent_builds, parse_url, title):
<h1>%s</h1>
<p>These builds are for testing purposes only.</p>
</div>
- <ul>\n""" % title
+ <div id="builds">
+ <ul>\n""" % title
footer = """
- </ul>
+ </ul>
- <p>We have <a href="%s">more stuff</a> if you don't see what you're looking for.</p>
-
+ <p>We have <a href="%s">more stuff</a> if you don't see what you're looking for.</p>
+ </div>
</body>
</html>""" % parse_url
+ extension = ""
middle = ""
for build in recent_builds:
- middle += '\n<li class="' + build['css_class'] + '">\n'
+ middle += '\n<li class="' + build['css_class'] + '"'
+ if build['extension'] != extension:
+ middle += 'style="clear: both;"'
+ extension = build['extension']
+ middle += '>\n'
middle += '<a href="' + build['url'] + '">'
middle += build['name']
middle += '</a>'
- middle += ' ' + build['size'] + 'B'
+ middle += '' + build['size'] + 'B'
middle += ' ' + build['extension']
middle += '<br>\n'
middle += '<small>Built on ' + build['date'] + '</small>\n'
@@ -145,8 +149,11 @@ def getRecentBuilds(builds):
for build in builds:
if build['url'].endswith(f.suffix + "." + f.extension) and f.css_class not in recent_builds_dict:
recent_builds_dict[f.css_class] = build
- items = recent_builds_dict.items()
- items.sort()
+ items = []
+ for f in files_wanted:
+ if recent_builds_dict.has_key(f.css_class):
+ items.append((f.css_class, recent_builds_dict[f.css_class]))
+
return [value for key, value in items]
def generateBuildPages(output_path, parse_url, dest_page, title):
@@ -173,7 +180,8 @@ def main():
for page in build_pages:
generateBuildPages(options.output_path, page["url"], page["html"], page["title"])
- copyFile(options.output_path, "firefox.png")
+ copyFile(options.output_path, "nightly.png")
+ copyFile(options.output_path, "blueGradient.png")
copyFile(options.output_path, "nightly.css")
if __name__ == '__main__':
@cvan

you used #333 above. for consistency's and brevity's sakes, this should also be #333.

@cvan

0 0 is unnecessary

@tofumatt

This is three years old and already committed. It's probably outdated. File a pull request if you have issues, please don't code review ancient code.

Please sign in to comment.
Something went wrong with that request. Please try again.