 Issue 29322769:
  Issue 2844 - Create Microsoft Edge coming soon landing page  (Closed)
    
  
    Issue 29322769:
  Issue 2844 - Create Microsoft Edge coming soon landing page  (Closed) 
  | Index: static/css/index.css | 
| =================================================================== | 
| --- a/static/css/index.css | 
| +++ b/static/css/index.css | 
| @@ -69,7 +69,7 @@ | 
| display: inline-block; | 
| } | 
| -#video, #maxthon-instruction | 
| +#video, #maxthon-instruction, #edge-teaser | 
| { | 
| background: #ffffff; | 
| padding: 10px; | 
| @@ -80,7 +80,7 @@ | 
| #maxthon-instruction, | 
| html[lang^="zh_"] #content.maxthon #video, | 
| #content.maxthon #feature-free, | 
| -#feature-maxthon | 
| +#feature-maxthon, #edge-subscription, #edge-teaser | 
| { | 
| display: none; | 
| } | 
| @@ -96,6 +96,16 @@ | 
| display: inline-block; | 
| } | 
| +#content.edge #video | 
| +{ | 
| + display: none; | 
| +} | 
| + | 
| +#content.edge #edge-subscription | 
| +{ | 
| + display: block; | 
| +} | 
| + | 
| #summary h1, #summary h2 | 
| { | 
| font-weight: normal; | 
| @@ -417,6 +427,16 @@ | 
| background-position: -128px -109px; | 
| } | 
| +#platform-edge | 
| +{ | 
| + background-position: -68px -171px; | 
| +} | 
| + | 
| +#content.edge #platform-edge | 
| +{ | 
| + background-position: -68px -151px; | 
| +} | 
| + | 
| .warning-platform, | 
| .disclaimer, | 
| #maxthon-notification | 
| @@ -609,7 +629,8 @@ | 
| #more-container.internet-explorer #tab-internet-explorer, | 
| #more-container.safari #tab-safari, | 
| #more-container.yandex-browser #tab-yandex-browser, | 
| -#more-container.maxthon #tab-maxthon | 
| +#more-container.maxthon #tab-maxthon, | 
| +#more-container.edge #tab-edge | 
| { | 
| display: inline-block; | 
| } | 
| @@ -622,6 +643,80 @@ | 
| #more-container.safari #more-safari, | 
| #more-container.yandex-browser #more-yandex-browser, | 
| #more-container.maxthon #more-maxthon | 
| +#more-container.edge #more-edge | 
| { | 
| background-color: #dbd4c7; | 
| } | 
| +#content h3.subscribe-header | 
| 
saroyanm
2015/09/24 14:52:04
Detail: I think we can move rules that are related
 | 
| +{ | 
| + font-size: 18px; | 
| + margin-bottom: 0px; | 
| +} | 
| + | 
| +#edge-subscription h3 | 
| +{ | 
| + margin: 19px 0px 10px 0px; | 
| + color: #2d79c0; | 
| + font-weight: bold; | 
| + font-family: Arial; | 
| + font-size: 14px; | 
| +} | 
| + | 
| +#edge-subscription | 
| +{ | 
| + margin: 0px 0px 10px 40px; | 
| +} | 
| + | 
| +html[dir="rtl"] #edge-subscription | 
| +{ | 
| + margin: 0px 40px 10px 0px; | 
| +} | 
| + | 
| +#edge-subscription input[type=email] | 
| +{ | 
| + border: solid 2px #a5a5a5; | 
| + margin-right: 0px; | 
| 
saroyanm
2015/09/24 14:52:04
Why do we need this rule ?
I can't see any use for
 | 
| + padding: 0px 10px; | 
| + font-size: 14px; | 
| + font-family: Arial; | 
| + line-height: 36px; | 
| + height: 36px; /* IE 8 */ | 
| +} | 
| + | 
| +#edge-subscription input[type=email].placeholder | 
| +{ | 
| + color: #a5a5a5; | 
| +} | 
| + | 
| +#edge-subscription button[type=submit] | 
| +{ | 
| + border: solid 1px #357EBD; | 
| + border-radius: 3px; | 
| + background-color: #428BCA; | 
| 
saroyanm
2015/09/24 14:52:04
Detail: make sense to use consistent HEX numbers,
 | 
| + color: #FFFFFF; | 
| + font-size: 14px; | 
| + font-weight: bold; | 
| + width: 146px; | 
| + height: 40px; | 
| + margin-left: 0px; | 
| 
saroyanm
2015/09/24 14:52:03
Same goes for this rule ? If there is no need to h
 | 
| +} | 
| + | 
| + | 
| +#edge-subscription form .sucess-label, #edge-subscription form .response-error, | 
| +#edge-subscription form .invalid-label, #edge-subscription form.success input, | 
| +#edge-subscription form.success button, #other-browsers | 
| +{ | 
| + display: none; | 
| +} | 
| + | 
| +#edge-subscription form .invalid-label | 
| +{ | 
| + margin-top: 15px; | 
| +} | 
| + | 
| +#edge-subscription form.success .sucess-label, | 
| +#edge-subscription form.invalid .invalid-label, | 
| +#edge-subscription form.error .response-error | 
| +{ | 
| + display: block; | 
| +} |