#1568: [patch] CharFields with small maxlength are still displayed with a size
of
30
--------------------------------------+-------------------------------------
Reporter: [EMAIL PROTECTED] | Owner: adrian
Status: closed | Component: Admin interface
Version: SVN | Resolution: wontfix
Keywords: | Stage: Ready for checkin
Has_patch: 1 | Needs_docs: 0
Needs_tests: 0 | Needs_better_patch: 0
--------------------------------------+-------------------------------------
Changes (by mtredinnick):
* status: new => closed
* resolution: => wontfix
Comment:
The oldforms package has been end-of-lifed now, so we are not going to be
applying any further enhancements like this to it. We do want to check
that the same problem doesn't appear on newforms, though.
[For the record, there is a problem with this patch as it stands: it
should only trim the length for really short fields. Otherwise, a field
with a maxlength of 200 will destroy the form layout. So some kind of
check that it only adjusts the length to maxlength if maxlength < 30 would
be required. That is why self.length exists in the first place. It should
also probably have had a minimum of 2 or 3 on the length.]
--
Ticket URL: <http://code.djangoproject.com/ticket/1568#comment:4>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" 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/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
