Opened 16 years ago
Closed 16 years ago
#10860 closed (invalid)
Invalid paragraph tag in comments preview template
Reported by: | justquick | Owned by: | nobody |
---|---|---|---|
Component: | contrib.comments | Version: | dev |
Severity: | Keywords: | comments html | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
A single <p>
tag in comments/preview.html was not closed. It is a very small problem, im just being a perfectionist. Thanks so much for this great framework!
Attachments (1)
Change History (5)
by , 16 years ago
Attachment: | preview.html.diff added |
---|
comment:1 by , 16 years ago
There are 3 <p> tags in here, all of them appear to have been closed: http://code.djangoproject.com/browser/django/trunk/django/contrib/comments/templates/comments/preview.html . Can you show where you see the unclosed one?
comment:2 by , 16 years ago
The patch is pretty clear which one it is, but the one im looking at is on line 24 and its the only html on that line.
comment:3 by , 16 years ago
Just looked at the form.html as well http://code.djangoproject.com/browser/django/trunk/django/contrib/comments/templates/comments/form.html. Same problem, line 9. This made me think that possibly there were others in comments templates, so i just checked and its only in form.html and preview.html. Feel free to double check me (grep '<p' django/contrib/comments/templates/comments/*.html)
comment:4 by , 16 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
No wait, I was not lookin at this right. False alarm, sorry guys
svn diff output