Major updates to form CSS. Now using unordered lists for layouts. Added .gInline for forms like login, search, tags. Updated comment, login, search, and tagging forms. All form styles live in screen.css now. Still a lot to do, but this is a good foundation.

4 files changed, 31 insertions, 27 deletions

diff --git a/modules/comment/views/comment_form.html.php b/modules/comment/views/comment_form.html.php
index 163fd180..1561035f 100644
--- a/modules/comment/views/comment_form.html.php
+++ b/modules/comment/views/comment_form.html.php
@@ -1,21 +1,25 @@
 <? defined("SYSPATH") or die("No direct script access."); ?>
-<form id="gCommentAdd" class="gExpandedForm">
+<form id="gCommentAdd">
   <fieldset>
     <legend>Add comment</legend>
-    <div class="row">
-      <label for="gCommentAuthor"><?= _("Your Name") ?></label>
-      <input type="text" name="author" id="gCommentAuthor" class="text" />
-    </div>
-    <div class="row">
-      <label for="gCommentEmail"><?= _("Your Email (not displayed)") ?></label>
-      <input type="text" name="email" id="gCommentEmail" class="text" />
-    </div>
-    <div class="row">
-      <label for="gCommentText"><?= _("Comment") ?></label>
-      <textarea name="text" id="gCommentText"></textarea>
-    </div>
-    <input type="hidden" id="gItemId" name="item_id" value="<?= $item_id ?>" />
-    <input type="submit" id="gCommentSubmit" value="<?= _("Add") ?>" />
+    <ul>
+      <li>
+        <label for="gCommentAuthor"><?= _("Your Name") ?></label>
+        <input type="text" name="author" id="gCommentAuthor" />
+      </li>
+      <li>
+        <label for="gCommentEmail"><?= _("Your Email (not displayed)") ?></label>
+        <input type="text" name="email" id="gCommentEmail" />
+      </li>
+      <li>
+        <label for="gCommentText"><?= _("Comment") ?></label>
+        <textarea name="text" id="gCommentText"></textarea>
+      </li>
+      <li>
+        <input type="hidden" id="gItemId" name="item_id" value="<?= $item_id ?>" />
+        <input type="submit" id="gCommentSubmit" value="<?= _("Add") ?>" />
+      </li>
+    </ul>
   </fieldset>
 </form>
 
diff --git a/modules/comment/views/comment_list.html.php b/modules/comment/views/comment_list.html.php
index 1545a106..ea824597 100644
--- a/modules/comment/views/comment_list.html.php
+++ b/modules/comment/views/comment_list.html.php
@@ -1,11 +1,11 @@
 <? defined("SYSPATH") or die("No direct script access."); ?>
 <ul id="gCommentThread">
   <? foreach (array_reverse($comments) as $index => $comment): ?>
-  <li id="gComment-<?= $index; ?>" class="gComment <?= $index % 2 ? 'odd' : 'even' ?>">
+  <li id="gComment-<?= $index; ?>" class="gComment <?= $index % 2 ? 'gOdd' : 'gEven' ?>">
     <p>
       <a href="#" class="gAuthor"><?= $comment->author ?></a>
       <?= comment::format_elapsed_time($comment->datetime) ?>,
-      <span class="understate"><?= strftime('%c', $comment->datetime) ?></span>
+      <span class="gUnderstate"><?= strftime('%c', $comment->datetime) ?></span>
     </p>
     <div>
       <?= $comment->text ?>
diff --git a/modules/tags/views/tags_block.html.php b/modules/tags/views/tags_block.html.php
index 40427ee9..f23874ff 100644
--- a/modules/tags/views/tags_block.html.php
+++ b/modules/tags/views/tags_block.html.php
@@ -38,7 +38,9 @@
 </ul>
 
 <form id="gAddTag">
-  <input type="text" class="text" value="add new tags ..." id="newtags" />
-  <input type="submit" class="submit" value="add" />
-  <label for="newtags" class="understate">(separated by commas)</label>
+  <ul class="gInline">
+    <li><input type="text" class="text" value="add new tags..." id="gNewTags" /></li>
+    <li class="gNoLabels"><input type="submit" value="add" /></li>
+  </ul>
+  <label for="gNewTags" class="gUnderState">(separated by commas)</label>
 </form>
diff --git a/modules/user/views/login.html.php b/modules/user/views/login.html.php
index fb770f3f..fa7dbc9b 100644
--- a/modules/user/views/login.html.php
+++ b/modules/user/views/login.html.php
@@ -2,22 +2,20 @@
 <form id="gLogin" action="<?= url::site("login/process") ?>">
   <fieldset>
     <legend>Login</legend>
-    <ul>
+    <ul class="gInline">
       <li>
-        <label for="username">Username</label>
+        <label for="gUsername">Username</label>
         <input type="text" id="gUsername" />
       </li>
       <li>
-        <label for="password">Password</label>
+        <label for="gPassword">Password</label>
         <input type="password" id="gPassword" />
       </li>
       <li>
-        <input type="submit" class="submit" value="<?= _("Login")?>" />
+        <input type="submit" value="<?= _("Login")?>" />
       </li>
     </ul>
-  </fieldset>
-  <fieldset>
-    <div class="gStatus gError gDisplayNone" id="gLoginMessage">
+    <div id="gLoginMessage" class="gStatus gError gDisplayNone">
     </div>
   </fieldset>
 </form>