From e155c98f9e859b1d6e9c9a6be763fb12f8b4c26c Mon Sep 17 00:00:00 2001
From: Artyom <yom@artyom.me>
Date: Fri, 26 Feb 2016 01:31:24 +0300
Subject: [PATCH] More styling

---
 src/Main.hs    | 52 +++++++++++++++++++++++++++-----------------------
 static/css.css | 20 ++++++++++---------
 2 files changed, 39 insertions(+), 33 deletions(-)

diff --git a/src/Main.hs b/src/Main.hs
index efce3ef..2527d8d 100644
--- a/src/Main.hs
+++ b/src/Main.hs
@@ -329,9 +329,12 @@ renderRoot globalState = do
     Here are some guidelines/observations/etc that probably make sense:
 
       * sort pros/cons by importance
+
       * if you don't like something for any reason, edit it
+
       * if you're unsure about something, still write it
         (just warn others that you're unsure)
+
       * if you have useful information of any kind that doesn't fit,
         add it to the category notes
 
@@ -428,7 +431,7 @@ renderItemInfo editable item =
   div_ $ do
     this <- thisNode
     case editable of
-      Editable -> h3_ $ do
+      Editable -> span_ [style_ "font-size:150%"] $ do
         -- If the library is on Hackage, the title links to its Hackage page;
         -- otherwise, it doesn't link anywhere. Even if the link field is
         -- present, it's going to be rendered as “(site)”, not linked in the
@@ -472,33 +475,34 @@ renderItemTraits :: Editable -> Item -> HtmlT IO ()
 renderItemTraits editable item =
   div_ [class_ "traits"] $ do
     this <- thisNode
+    div_ [class_ "traits-groups-container"] $ do
+      div_ [class_ "traits-group"] $ do
+        p_ "Pros:"
+        case editable of
+          Normal ->
+            ul_ $ mapM_ (renderTrait Normal (item^.uid)) (item^.pros)
+          Editable -> do
+            listNode <- ul_ $ do
+              mapM_ (renderTrait Editable (item^.uid)) (item^.pros)
+              thisNode
+            textInput [placeholder_ "add pro"] $
+              JS.addPro (listNode, item^.uid, inputValue) <> clearInput
+      div_ [class_ "traits-group"] $ do
+        p_ "Cons:"
+        case editable of
+          Normal ->
+            ul_ $ mapM_ (renderTrait Normal (item^.uid)) (item^.cons)
+          Editable -> do
+            listNode <- ul_ $ do
+              mapM_ (renderTrait Editable (item^.uid)) (item^.cons)
+              thisNode
+            textInput [placeholder_ "add con"] $
+              JS.addCon (listNode, item^.uid, inputValue) <> clearInput
     case editable of
-      Normal -> textButton "edit" $
+      Normal -> textButton "edit pros/cons" $
         JS.setItemTraitsMode (this, item^.uid, Editable)
       Editable -> textButton "edit off" $
         JS.setItemTraitsMode (this, item^.uid, Normal)
-    div_ [class_ "traits-group"] $ do
-      p_ "Pros:"
-      case editable of
-        Normal ->
-          ul_ $ mapM_ (renderTrait Normal (item^.uid)) (item^.pros)
-        Editable -> do
-          listNode <- ul_ $ do
-            mapM_ (renderTrait Editable (item^.uid)) (item^.pros)
-            thisNode
-          textInput [placeholder_ "add pro"] $
-            JS.addPro (listNode, item^.uid, inputValue) <> clearInput
-    div_ [class_ "traits-group"] $ do
-      p_ "Cons:"
-      case editable of
-        Normal ->
-          ul_ $ mapM_ (renderTrait Normal (item^.uid)) (item^.cons)
-        Editable -> do
-          listNode <- ul_ $ do
-            mapM_ (renderTrait Editable (item^.uid)) (item^.cons)
-            thisNode
-          textInput [placeholder_ "add con"] $
-            JS.addCon (listNode, item^.uid, inputValue) <> clearInput
 
 renderTrait :: Editable -> Uid -> Trait -> HtmlT IO ()
 renderTrait Normal _itemId trait = li_ (renderMarkdownLine (trait^.content))
diff --git a/static/css.css b/static/css.css
index 88236f3..f5bfdaf 100644
--- a/static/css.css
+++ b/static/css.css
@@ -13,18 +13,20 @@ body {
 .category {
   margin-top: 3em; }
 
-.items {
+.item {
+  background-color: #f0f0f0;
+  margin-top: 20px;
+  padding: 10px 15px; }
+
+.traits-groups-container {
   display: flex; }
 
-.item {
-  border: 1px solid;
-  margin: 5px; }
-
-.traits {
-  display: flex }
-
 .traits-group {
-  margin: 0px 5px; }
+  margin: 0px 5px;
+  flex: 1; }
+
+.traits-group > ul {
+  padding-left: 1em; }
 
 .traits-group > ul li {
   margin: 10px 0px; }