Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#5695 closed (fixed)

[newform-admin] - i18n and xhtml problem in AdminFileWidget

Reported by: Petr Marhoun <petr.marhoun@…> 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)

admin-file-widget.diff (689 bytes ) - added by Petr Marhoun <petr.marhoun@…> 16 years ago.
5695.diff (1.4 KB ) - added by Marc Garcia 16 years ago.
Improved patch

Download all attachments as: .zip

Change History (8)

by Petr Marhoun <petr.marhoun@…>, 16 years ago

Attachment: admin-file-widget.diff added

comment:1 by Brian Rosner, 16 years ago

Triage Stage: UnreviewedReady for checkin

Looks good to me. Marking as Ready for checkin.

comment:2 by Marc Garcia, 16 years ago

Cc: marc.garcia@… added
Component: InternationalizationAdmin 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

by Marc Garcia, 16 years ago

Attachment: 5695.diff added

Improved patch

comment:3 by Brian Rosner, 16 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 Brian Rosner, 16 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 Brian Rosner, 16 years ago

Resolution: fixed
Status: newclosed

(In [7212]) newforms-admin: Fixed #5695. AdminFileWidget now properly marks strings for translation and a minor XHTML fix.

comment:6 by Marc Garcia, 16 years ago

Cc: marc.garcia@… removed

Thanks for commiting.

Your ticket for target="_blank" is #6750

Note: See TracTickets for help on using tickets.
Back to Top