#5695 closed (fixed)
[newform-admin] - i18n and xhtml problem in AdminFileWidget
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | contrib.admin | Version: | newforms-admin |
| Severity: | Keywords: | nfa-blocker AdminFileWidget i18n invalid html | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Improvements introduced in [6457] are not internationalized and are not xhtml compatible.
Attachments (2)
Change History (8)
by , 18 years ago
| Attachment: | admin-file-widget.diff added |
|---|
comment:1 by , 18 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|
comment:2 by , 18 years ago
| Cc: | added |
|---|---|
| Component: | Internationalization → Admin interface |
| Keywords: | AdminFileWidget i18n invalid html added |
Previous patch improved:
- target="_blank" isn't valid in xhtml strict, changed to javascript
- Translatable labels not including colon
- Tests added
comment:3 by , 18 years ago
| Keywords: | nfa-blocker added |
|---|
This is pretty trivial and does need to be fixed before a merge to trunk. Tagging nfa-blocker.
comment:4 by , 18 years ago
I am going to fix this, but can the target="_blank" stuff be opened in a new ticket. I would rather deal with that separately.
comment:5 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:6 by , 18 years ago
| Cc: | removed |
|---|
Thanks for commiting.
Your ticket for target="_blank" is #6750
Note:
See TracTickets
for help on using tickets.
Looks good to me. Marking as Ready for checkin.