Issue 5740 - Implemented new abp.org footer styles

static/css/main.css

Index: static/css/main.css
===================================================================
--- a/static/css/main.css
+++ b/static/css/main.css
@@ -133,34 +133,16 @@
 .sprite
 {
   display: inline-block;
   font-size: 0px; /* fix for IE6 height bug */
   background-image: url(../img/sprite-main.png);
   background-repeat: no-repeat;
 }
 
-nav ul
-{
-  list-style: none;
-  margin: 0;
-  padding: 0;
-}
-
-nav a:link, nav a:visited
-{
-  color: #7d7d7d;
-  text-decoration: none;
-}
-
-nav a:hover
-{
-  text-decoration: underline;
-}
-
 header
 {
   display: block;
   position: relative;
   background: white;
   border: 1px solid #d9d9d9;
   border-top: none;
   box-shadow: 1px 1px 0 0 #d9d9d9;
@@ -264,88 +246,23 @@
   padding-left: 40px;
 }
 
 .toc li
 {
   margin: 4px 0;
 }
 
-#footer-main
-{
-  display: block;
-  background: white;
-  box-shadow: 0 -1px 0 0 #b5b4b0, 0 -2px 0 0 #e4e1dc;
-  padding: 0;
-  line-height: 1;
-}
-
-#footer-main nav h1, #footer-main nav li
-{
-  color: #444444;
-  border-bottom: 1px solid #dedede;
-}
-
-#footer-main h1
-{
-  font-size: 16px;
-  margin-top: 0;
-  margin-bottom: 15px;
-}
-
-#footer-main nav li
-{
-  margin-bottom: 5px;
-}
-
-#social-list
+#social-list ul
 {
   list-style: none;
   padding: 0;
   margin: 0;
 }
 
-.social-entry
-{
-  display: inline-block;
-  margin: 0px 5px;
-}
-
-.social-entry a
-{
-  width: 82px;
-  height: 82px;
-  -webkit-transition: opacity .5s ease;
-  -moz-transition: opacity .5s ease;
-  -ms-transition: opacity .5s ease;
-  -o-transition: opacity .5s ease;
-  transition: opacity .5s ease;
-  opacity: .6;
-}
-
-.social-entry a:hover
-{
-  opacity: 1.0;
-}
-
-#social-facebook
-{
-  background-position: 0px 0px;
-}
-
-#social-twitter
-{
-  background-position: 0px -83px;
-}
-
-#social-gplus
-{
-  background-position: -83px 0px;
-}
-
 div.animation[started="false"]:before
 {
   content: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAA8AAAAPBAMAAADJ%2BIh5AAAAFVBMVEUAgAAAgAD%2F%2F%2F%2Fr9uvY7Ni%2B376h0aGeNpcsAAAAAXRSTlNtc%2BXFgwAAADVJREFUeF5dzVEKACAMAlC7QW6t%2F7r%2FIYOBEPr1GOgw2AHiduaHkwKXwBK4BYZdylrpy4K%2BP%2Fu5C1CVKVSzAAAAAElFTkSuQmCC);
   padding-left: 0px;
   padding-right: 3px;
 }
 
 html[dir="rtl"] div.animation[started="false"]:before
@@ -377,8 +294,161 @@
 {
   border-left: 5px solid #d14841;
 }
 
 [dir="rtl"] .alert
 {
   border-right: 5px solid #d14841;
 }
+
+@media(max-width: 767px)
+{
+  #site-footer .column
+  {
+    display: none;
+  }
+
+  #site-footer #social-list
+  {
+    display: block;
+  }
+
+  #site-footer #social-list ul
+  {
+    margin: 0em 1em;
+  }
+}
+
+#site-footer
+{
+  margin-top: 2em;
+  padding: 2em 0em;
+  color: #ececec;
+  background-color: #292929;
+  overflow: auto;
+}
+
+#site-footer h5

[juliandoucette, 2017/10/06 12:57:58]

Note: h5 will inherit color from #site-footer and margin/font-size from global
styles. As a result, I will address this (the site-wide h5 style) separately.

[ire, 2017/10/09 10:37:59]

Acknowledged.

+{
+  color: #ececec;
+  font-size: 22px;
+  margin: 16px 0px 40px 0px;
+}
+
+#site-footer h5::after
+{
+  content: "";
+  display: block;
+  width: 30px;

[juliandoucette, 2017/10/06 12:58:01]

Note: I used em for this width in the new implementation. This is a little
unintuitive because you would think that 2em = 2 letters width. But this is not
true because this is not a monospace font. I eyeballed this width based on an h5
font-size of 22px.

[ire, 2017/10/09 10:38:00]

Acknowledged.

+  height: 3px;
+  margin: 16px 0px 0px 0px;
+  border: none;
+  background-color: #ececec;
+}
+
+#site-footer p

[juliandoucette, 2017/10/06 12:57:58]

Note: I will inherit this font-size from #site-footer instead.

[ire, 2017/10/09 10:38:00]

Acknowledged.

+{
+  font-size: 18px;
+}
+
+#site-footer li

[juliandoucette, 2017/10/06 12:57:58]

Note: I would like to inherit this style from reset.css. But I think that is
out-of-scope for this change.

[ire, 2017/10/09 10:38:00]

Acknowledged.

+{
+  list-style-type: none;
+}
+
+#site-footer ul
+{
+  padding: 0px;
+}
+
+#site-footer a
+{
+  font-size: 18px;

[juliandoucette, 2017/10/06 12:57:59]

Note: I will inherit this font-size from #site-footer.

[ire, 2017/10/09 10:38:00]

Acknowledged.

+  color: #ececec;
+  text-decoration: none;
+}
+
+#site-footer a:hover,
+#site-footer a:active,
+#site-footer a:focus
+{
+  color: #fff;

[juliandoucette, 2017/10/06 12:57:59]

Note: This is indistinguishable. But it's better than nothing IMO (this change
was improvised/unspecified).

[ire, 2017/10/09 10:37:59]

I agree

+  text-decoration: underline;
+}
+
+#footer-legal
+{
+  margin-top: 3em;

[juliandoucette, 2017/10/06 12:57:58]

Note: This was improvised (Paul said that he made a similar design, and approved
my spacing). Anyway, I thought 3em was a little too much. So I changed it to
2em.

[ire, 2017/10/09 10:38:00]

Acknowledged.

+}
+
+#site-footer #copyright-notice,
+#site-footer #copyright-notice a,
+#legal-list a
+{
+  font-size: 14px;
+}
+
+@media(min-width: 768px)
+{
+  #site-footer #copyright-notice,
+  #site-footer #copyright-notice a,
+  #legal-list a
+  {
+    font-size: 16px;
+  }
+}
+
+#social-list li,

[juliandoucette, 2017/10/06 12:58:01]

Note: I separated these because they are separate ~components. I am choosing
readability over performance in that way. I don't have a strong opinion about
this case in particular.

+#legal-list li
+{
+  display: block;
+  float: left;

[juliandoucette, 2017/10/06 12:58:00]

Note: I forgot to support rtl here. I'm not really sure of the value of
reversing the order when rtl. But I will anyway.

[ire, 2017/10/09 10:37:59]

Acknowledged.

+}
+
+#social-list li
+{
+  margin-right: 10px;

[juliandoucette, 2017/10/06 12:57:58]

Note: This width wasn't specified, but it looks larger in the mockups. I made it
smaller so that I could fit all of these glyphs on one line on smaller screens.
Of course, I could use a media query or a % width instead. But I won't unless
you think it's necessary.

[ire, 2017/10/09 10:38:00]

I don't think it is necessary. IMO the spacing is good and readable

[juliandoucette, 2017/10/11 13:12:05]

Acknowledged.

+}
+
+[dir="rtl"] #social-list li
+{
+  margin-left: 10px;
+}
+
+#social-list img
+{
+  height: 40px;

[juliandoucette, 2017/10/06 12:58:00]

Note: 50px was specified, but 50px made it too hard to fit these all on one line
(especially on smaller screens). I chose 40px (2em) instead.

[ire, 2017/10/09 10:37:59]

Acknowledged.

+}
+
+#legal-list li::after
+{
+  margin: 0px 5px;
+  content: "|"
+}
+
+#legal-list li:last-of-type::after

[juliandoucette, 2017/10/06 12:58:00]

Note: I'm being a little tricky here using ::after (IE9+) and :last-of-type
(IE9+) for progressive enhancement (the "|" will not show at all lt IE9)

[ire, 2017/10/09 10:37:59]

Love it!

+{
+  content: none;
+}
+
+@media(min-width: 1024px)

[juliandoucette, 2017/10/06 12:57:58]

Note: I made this breakpoint larger [to match the largest grid breakpoint, to
avoid clearing if the copyright notice is longer when translated]

[ire, 2017/10/09 10:38:00]

Acknowledged.

+{
+  #copyright-notice
+  {
+    float: left;
+  }
+
+  [dir="rtl"] #copyright-notice
+  {
+    float: right;
+  }
+
+  #legal-list
+  {
+    float: right;
+  }
+
+  [dir="rtl"] #legal-list
+  {
+    float: left;
+  }
+}