Two patches for the welcome app.

The CSS patch makes a clearer distinction between the navigation  
column and the main column. Without it, the "Hello World" text tends  
to visually run into "Authentication". The border is visually  
consistent with the rest of the page design, and IMO looks pretty good.

It also replaces the // comment markers (illegal in CSS) with /* ... */

The view patch does two things.

1. Replace the explicit <br> after the <h2> with a P(_style)  
attribute; we shouldn't be using <br> for layout spacing.

2. Use A() for the links, which fixes the missing </a> tags.

The resulting page is now valid XHTML.


===================================================================
--- base.css    (revision 1094)
+++ base.css    (working copy)
@@ -204,6 +204,10 @@
      border-top: 1px #99f solid;
  }

+.column1 {
+    border-right: 1px #99f solid;
+}
+
  .width10em {
      width: 10em;
      background: transparent;
@@ -415,9 +419,11 @@
  ul.web2py-menu li.web2py-menu-expand li.web2py-menu-expand li.web2py- 
menu-expand a
  {
      padding-right: 25px;
-    // background-image: url("expand-right.gif");
-    // background-repeat: no-repeat;
-    // background-position: 100% 50%;
+    /*
+    background-image: url("expand-right.gif");
+    background-repeat: no-repeat;
+    background-position: 100% 50%;
+    */
  }

  ul.web2py-menu-vRight li.web2py-menu-expand a,
@@ -429,17 +435,21 @@
  {
      padding-right: 5px;
      padding-left: 20px;
-    // background-image: url("expand-left.gif");
-    // background-repeat: no-repeat;
-    // background-position: -5px 50%;
+    /*
+    background-image: url("expand-left.gif");
+    background-repeat: no-repeat;
+    background-position: -5px 50%;
+    */
  }

  ul.web2py-menu-horizontal li.web2py-menu-expand a
  {
      padding-left: 5px;
      padding-right: 15px;
-    // background-image: url("expand-down.gif");
-    // background-position: 100% 50%;
+    /*
+    background-image: url("expand-down.gif");
+    background-position: 100% 50%;
+    */
  }

  ul.web2py-menu li.web2py-menu-expand li a,


===================================================================
--- index.html  (revision 1094)
+++ index.html  (working copy)
@@ -1,6 +1,6 @@
  {{extend 'layout.html'}}

-{{try:}}{{=H2(message)}}{{except:}}{{=BEAUTIFY(response._vars)}} 
{{pass}}<br />
+{{try:}}{{=H2(message)}}{{except:}}{{=BEAUTIFY(response._vars)}} 
{{pass}}

-<a href="{{=URL('admin','default','index')}}">{{=T("click here for  
the administrative interface")}}<br />
-<a href="{{=URL('examples','default','index')}}">{{=T("click here for  
online examples")}}
+{{=P(A(T("click here for the administrative interface"),  
_href=URL('admin','default','index')),_style="padding-top:1em;")}}
+{{=P(A(T("click here for online examples"),  
_href=URL('examples','default','index')))}}


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"web2py Web Framework" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/web2py?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to