Merge lp:~claire-couchone/sahana-eden/branch into lp:sahana-eden

[Claire McCabe]

So, we're adding a tool tip, mmmkay?

[Claire McCabe (claire-couchone)]

Um....so, basically, we added something so that people can have a default date of birth in the sahara eden. Um...yeah.

[Fran Boon (flavour)]

Thanks Claire,

#474 was, I think, requesting a little calendar icon next to the field which would provide a visual indicator that this was a date field which would bring up a date picker widget.
This could easily be added to the .comment for the field, although there should then be more work to make the icon clickable to bring up the widget (otherwise I can see a UX bug being reported for this)

I don't see any code to default the date of birth field.
I'm also not sure what the purpose of such would be - can you elaborate?

NB There shouldn't be any need to import datetime within this model since it has already been imported earlier in the request processing - in 00_db.py.

PS This merge is against a slightly older version of Sahana Eden so it still used Tstr(). This has now been deprecated since Web2Py has extended T() to support the common str() operations such as concatenation.

Best Wishes,
Fran.

review: Needs Fixing

Unmerged revisions

1261. By Sahana Eden <sahana@sahana-dev> on 2010-10-01

Ticket # 474 we've fixed the date of birth issue. Ie, we added a tooltip.

=== modified file 'models/02_pr.py'
--- models/02_pr.py	2010-09-28 07:58:41 +0000
+++ models/02_pr.py	2010-10-01 02:46:45 +0000
@@ -290,6 +290,7 @@
 # -----------------------------------------------------------------------------
 resource = "person"
 tablename = "%s_%s" % (module, resource)
+import datetime
 table = db.define_table(tablename,
                         pe_id(),
                         pe_label(),
@@ -323,7 +324,8 @@
 
 table.first_name.requires = IS_NOT_EMPTY()
 table.first_name.requires.error_message = T("Please enter a First Name")
-
+table.date_of_birth.comment = DIV(DIV(_class="tooltip",
+    _title=Tstr("ID Label") + "|" + Tstr("Click in the field for frickin' calendar options.")))
 table.pe_label.comment = DIV(DIV(_class="tooltip",
     _title=T("ID Label") + "|" + T("Number or Label on the identification tag this person is wearing (if any).")))
 table.first_name.comment =  DIV(SPAN("*", _class="req", _style="padding-right: 5px;"), DIV(_class="tooltip",