#7028: Better raw_id_fields feedback in newform-admins branch ---------------------------------------------------+------------------------ Reporter: Marcob <marcob...@gmail.com> | Owner: nobody Status: new | Milestone: 1.2 Component: django.contrib.admin | Version: SVN Resolution: | Keywords: raw-id-fields nfa-someday Stage: Accepted | Has_patch: 1 Needs_docs: 0 | Needs_tests: 0 Needs_better_patch: 0 | ---------------------------------------------------+------------------------ Comment (by mrts):
Fixed with {{{.replace("'", "`")}}} as was originally intended (but incorrectly implemented). I wonder if this is causes problems for Romanic languages that use {{{`}}} for accents -- Marco, is it OK for Italian? I.e. you would see ''Servizi connessi all{{{`}}}Agricoltura'' instead of ''Servizi connessi all'Agricoltura'' in the object link. Also, I wonder if English speakers would be annoyed by seeing ''Aesop{{{`}}}s fables'' instead of ''Aesop's fables''. Other fixes: * more than 7 words are now truncated instead of the somewhat arbitrary 14 in the original patch. Even retaining 7 may be a few too many, * proper XSS protection with `nameElem.innerHTML = ... html_escape(chosenName) ...;`, also get rid of the ugliness mentioned a few comments above * add `?_popup=1` to the related object popup and dismiss on save * consistent label usage. Will upload the patch tomorrow. -- Ticket URL: <http://code.djangoproject.com/ticket/7028#comment:26> Django <http://code.djangoproject.com/> The Web framework for perfectionists with deadlines. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django updates" group. To post to this group, send email to django-updates@googlegroups.com To unsubscribe from this group, send email to django-updates+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/django-updates?hl=en -~----------~----~----~----~------~----~------~--~---