PR#633: Update the style of start.fedoraproject.org - fedora-websites

fedora-websites

#633 Update the style of start.fedoraproject.org

Merged 7 years ago by ryanlerch. Opened 7 years ago by mhsabbagh.

Unknown source master into master

Update start.fedoraproject.org/static/css/app.css

mhsabbagh • 7 years ago

8a2211f

Update start.fedoraproject.org/static/css/app.css

mhsabbagh • 7 years ago

75a8760

start.fedoraproject.org/static/css/app.css

file modified

+12 -15

		`@@ -6,10 +6,10 @@`
		`}`

		`.grey {`
		`- background-color: #58595b;`
		`+ background-color: #DEDEDE;`
		`margin: 0 auto;`
		`width: 100%;`
		`- color: #fff;`
		`+ color: #555;`
		`}`

		`.product-header {`
		`@@ -17,7 +17,7 @@`
		`}`

		`.bluebox {`
		`- background-color: #3c6eb4;`
		`+ background-color: #3c6eb4;`
		`height: 85px;`
		`}`

		`@@ -37,7 +37,7 @@`
		`/* Header doc */`

		`.border {`
		`- border-right: 2px solid #ccc;`
		`+ border-right: 2px solid rgba(255, 255, 255, 0.4);`
		`}`

		`.bluebox h3 {`
		`@@ -46,12 +46,12 @@`
		`}`

		`.bluebox a {`
		`- color: #ccc;`
		`+ color: #fff;`
		`text-decoration: none;`
		`}`

		`.bluebox a:hover {`
		`- color: #fff;`
		`+ color: #eee;`
		`text-decoration: none;`
		`}`

		`@@ -100,7 +100,7 @@`
		`border-radius: 5px;`
		`-moz-border-radius: 5px;`
		`-webkit-border-radius: 5px;`
		`- margin-right: 60px;`
		`+ margin-right: 30px;`
		`margin-bottom: -15px;`
		`padding: 5px 10px;`
		`}`
		`@@ -130,12 +130,14 @@`
		`}`

		`.comments {`
		`- background-color: #294172;`
		`- color: #fff;`
		`- padding: 5px 10px;`
		`+ color: #555;`
		`+ padding: 5px;`
		`width: 60px;`
		`float: right;`
		`font-size: 16px;`
		`+ border-radius: 5px;`
		`+ -webkit-border-radius: 5px;`
		`+ moz-border-radius: 5px;`
		`}`

		`.comment-icon {`
		`@@ -225,10 +227,6 @@`
		`/* special classes for small screens */`

		`@media (max-width: 767px) {`
		`- .border {`
		`- border-bottom: 2px solid #ccc;`
		`- border-right: none;`
		`- }`

		`.white img {`
		`width: 700px;`
		`@@ -256,4 +254,3 @@`
		`}`
		`}`
		`/* end small screens classes */`
		`-`

mhsabbagh commented 7 years ago

I have made the background darker and the header separators are back. Is it good?

mhsabbagh commented 7 years ago

Any comment? The requested changes are made. If something else needs to be changed. I can work on it.

ryanlerch commented 7 years ago

@mhsabbagh if this is a cosmetic change, it would be super useful here too to have a before / after screenshot here on this thread, so we can get a quick look at the changes.

cheers,
ryanlerch

mhsabbagh commented 7 years ago

@ryanlerch Hello.

This is the screenshot for the current page (before): http://i.imgur.com/XdSCGlR.png
This is the screenshot for the page after modification: http://i.imgur.com/coQL1ab.png

mhsabbagh commented 7 years ago

Follow-ups?

ryanlerch commented 7 years ago

@robyduck i like this lighter style. Any objections from your side?