Gauravwagh / django-grappelli

Automatically exported from code.google.com/p/django-grappelli
Other
0 stars 0 forks source link

Admin fields errror label - line.errors #318

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. If you set fieldset in admin eg. fieldsets = ((_('some label name'), 
{'fields': (('some_filed', 'some_other_field'),)}),)
2.Like you see above fields should be displayed one next to another.
3.

What is the expected output? What do you see instead?
Show error only in one field

What version of the product are you using? On what operating system?
current

Please provide any additional information below.
In file templates/admin/includes/fieldset.html.

If you set two fields one next to another, if one of them raise error,
user see errors in both fields what is defect. User should see errors only if 
field raise that error.

Original issue reported on code.google.com by lukaszga...@gmail.com on 18 Dec 2010 at 7:18

Attachments:

GoogleCodeExporter commented 9 years ago
reproduced.

already implemented the diff with my local repo. but it only solves half the 
issue.

@axel: please take a look at the fieldset-template. in line 6, the rows get an 
error-class (which is wrong if only one field within this row throws an error).

Original comment by sehmaschine on 18 Dec 2010 at 7:58

GoogleCodeExporter commented 9 years ago
Right, maybe this diff will be better.

Original comment by lukaszga...@gmail.com on 18 Dec 2010 at 10:11

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by sehmaschine on 23 Dec 2010 at 2:01

GoogleCodeExporter commented 9 years ago
solved with r1391
I compared the fieldset.htmls to the original django-templates and made some 
changes according to the patches and those original files.

testing is very much appreciated.

Original comment by parsch.inc@gmail.com on 3 Jan 2011 at 2:33