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
Bug 1133004. Fx family landing page. #2827
Conversation
861f497
to
d6f0513
Compare
34332bf
to
79b9a5c
Compare
Still to do:
The RTL work could be punted to post-launch since it doesn't look like it's yet localized in ar, fa, or he. |
'firefox_family': { | ||
'source_filenames': ( | ||
'css/sandstone/sandstone-resp.less', | ||
'css/firefox/family-nav.less', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope!
3381785
to
270d12f
Compare
var latestFirefoxVersionFull = $html.attr('data-latest-firefox'); | ||
|
||
// get latest full version (no point release info) for initial check | ||
var latestFirefoxVersion = parseInt(latestFirefoxVersionFull.split('.')[0], 10); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is needed? Looks like isFirefoxUpToDate()
already handles much of the logic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably not needed. I swiped this from firefox/new.js which has a lot of extra conditional bits of logic, but I think this is redundant for this page.
Let's not forget to show Tabzilla the door on this page also 👍 |
270d12f
to
c066003
Compare
6c16f2c
to
1b9b0af
Compare
Remove the old products page Update links in Tabzilla
1b9b0af
to
e26b51f
Compare
$('.product-list').on('click', 'a', function(e) { | ||
var newTab = ($(this).target === '_blank' || e.metaKey || e.ctrlKey); | ||
var href = $(this).attr('href'); | ||
var product = $(this).data('product'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this var isn't used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, should be passing the product name in place of the href. Fixed!
Aside from small JS nits above and 1400px wide intro/heading styles, things are looking good. |
IE9 crashes when calc() is used for background position Also reworked media queries to 1000px is the base width
e26b51f
to
edb9107
Compare
Looking good! r+ 🍝 |
Bug 1133004. Fx family landing page.
No description provided.