#8855 closed (duplicate)
Default comment form deficiencies
Reported by: | Jarek Zgoda | Owned by: | |
---|---|---|---|
Component: | contrib.comments | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Default comment form, as found in comments framework, suffers from some basic deficiencies:
- submit button labels ("Post" and "Preview") are not marked for translation;
- the
input
HTML elements (tags) are not closed, effectively prohibiting document validation as XHTML 1.0
Attached patch fixes these two problems and adds minor change in HTTP method declaration for consistency in style.
Attachments (1)
Change History (8)
by , 16 years ago
Attachment: | comments-form.html.diff added |
---|
comment:1 by , 16 years ago
milestone: | → 1.0 |
---|---|
Owner: | changed from | to
Status: | new → assigned |
I don't know what to do about the translation strings as we are in string freeze but the tags should be fixed.
comment:2 by , 16 years ago
Owner: | removed |
---|---|
Status: | assigned → new |
comment:3 by , 16 years ago
milestone: | 1.0 → post-1.0 |
---|
We'll fix this after 1.0 - can't add strings now.
comment:4 by , 16 years ago
I agree the XML errors should be fixed, since that could prevent the page from rendering. So I've done that in r8958. But only that. None of the style changes or anything. That can all wait.
comment:5 by , 16 years ago
Looking at default templates provided in comments framework I see there's much more places where such things can be found. As this can be effectively worked around using user-provided forms I agree this should wait for post-1.0.
As there are 2 or 3 different kinds of problems here, I'll create separate tickets for each of there problems (translations, XHTML compliance, style consistency) with appropriate patches. After that I'll ask for closing this ticket in favor of detailed tickets.
comment:6 by , 16 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Comment form fixes