Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
-+-
 Reporter:  Brian May|Owner:  Anton
 |  Samarchyan
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Brian May):

 I sent an update to the mailing list, which is awaiting moderation.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.42daec66a7966924d9cef8520258c90a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
-+-
 Reporter:  Brian May|Owner:  Anton
 |  Samarchyan
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Tim Graham):

 The consensus on the mailing list is toward deprecating the
 `DEFAULT_CONTENT_TYPE` setting. Do you have a use case that argues against
 that which you could add to the discussion, Brian?

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.409330827979fc04f8f5faf35aaafdf7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
-+-
 Reporter:  Brian May|Owner:  Anton
 |  Samarchyan
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * needs_better_patch:  0 => 1


Comment:

 I wrote to [https://groups.google.com/d/topic/django-
 developers/EdwwxxqcKVU/discussion django-developers] to try to get a
 consensus of whether or not we should make that change.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.4a3d462de596759b0f415d70340cd4a5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
-+-
 Reporter:  Brian May|Owner:  Anton
 |  Samarchyan
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Anton Samarchyan):

 * has_patch:  0 => 1
 * version:  1.7 => master


Comment:

 Added [https://github.com/django/django/pull/7807 PR]

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.5cce70048ad6bde62d88272ec9814c6a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
-+-
 Reporter:  Brian May|Owner:  Anton
 |  Samarchyan
 Type:  Bug  |   Status:  assigned
Component:  contrib.admin|  Version:  1.7
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Anton Samarchyan):

 * owner:  nobody => Anton Samarchyan
 * status:  new => assigned


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.7101e5e405067d0a7dd42fed356b47a7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  new
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+
Changes (by collinanderson):

 * cc: cmawebsite@… (added)


Comment:

 I'm personally not a huge fan of having every admin view manually specify
 the content type. Some crazy alternatives:
 1. Recommend against DEFAULT_CONTENT_TYPE as it's a magic global setting.
 The admin might be one example of a bigger issue.
 2. Make the admin XHTML compatible despite what we said earlier.
 https://github.com/django/django/pull/3824

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.1a1855a86f77c010632132b6f452c703%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  new
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+
Changes (by jarshwah):

 * status:  closed => new
 * resolution:  duplicate =>


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.7f3996a67a0e52a9fd6dd15a1866ef9d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+-
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+-
Changes (by jarshwah):

 * stage:  Unreviewed => Accepted


Comment:

 Reopening on the basis that the admin site breaks when the following
 option is set: DEFAULT_CONTENT_TYPE = "application/xhtml+xml"

 Since admin supports HTML 5 only - it should take measures to ensure the
 appropriate headers are sent when the default is not applicable.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.54d9af0eb4fcc5334ddfc10cd0632b1a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--

Comment (by brian):

 Yes, I believe that would be satisfactory.

 I just looked at the code to try and get a vague idea of how this could be
 done, looks like TemplateResponse instances would need changing in
 django/contrib/admin/options.py - if my quick glance is sufficient.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.4a4d31d744f67ce0b7cb6c4440e3bffa%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--

Comment (by jarshwah):

 In that case, would it maybe make sense to manually specify the
 mime/content type of all the admins HttpResponse objects? Would that avoid
 the problem you're seeing?

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.7fbe8f3ef039cb91935c640b9b3152d9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--

Comment (by brian):

 Django Admin doesn't need to support XHTML, however it should be possible
 to use it with applications that do use XHTML.

 As it is, it is not possible for applications to support both XHTML 5 and
 use Django Admin, because the DEFAULT_CONTENT_TYPE setting is global
 across all of Django. So you either have to use HTML or turn off Django
 Admin.

 Not to mention XHTML being superior because parsing errors are flagged
 immediately rather then being ignored.

 The meta setting I previously gave would allow Django admin to work as
 HTML even if the DEFAULT_CONTENT_TYPE setting has been changed, although
 it could perhaps be argued this is a bit of a kludge, as it overrides the
 HTTP headers.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.25af1e96fc1d21e0e84faf40445d7b1c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--

Comment (by jarshwah):

 I didn't get the implication that "HTML5 replaces XHTML" from the linked
 ticket. Just that Django has chosen to support HTML5 rather than XHTML
 [N].

 From your link: http://www.w3.org/TR/html5/introduction.html#html-vs-xhtml

 > The first such concrete syntax is the HTML syntax. This is the format
 suggested for most authors. ... This specification defines version 5.0 of
 the HTML syntax, known as "HTML 5".
 > The second concrete syntax is the XHTML syntax, which is an application
 of XML. ... This specification defines version 5.0 of the XHTML syntax,
 known as "XHTML 5".
 > The DOM, the HTML syntax, and the XHTML syntax cannot all represent the
 same content.

 My understanding is that authors should *choose* which to support. The
 authors of the Admin have chosen HTML 5. You are free to choose XHTML 5
 for your own templates.

 HTML 5 *is* a modern standard. There is no requirement (demonstrated) to
 support multiple, competing standards though.

 Is there a particular reason that you feel Django should support XHTML 5
 in the admin?

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.ecf059afda592426c9af1e450efa6a5b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--

Comment (by brian):

 My understanding is that XHTML is still a current standard, even with
 HTML5.

 See http://www.w3.org/TR/html5/, note the title "A vocabulary and
 associated APIs for HTML and XHTML". This was published last month.

 As such, the response in #19363 implying that HTML5 replaces XHTML is
 nonsense.

 Also it possible to add something like the following in the  section
 to force use of HTML, even when XHTML is the default:

 

 I feel that something like Django should really keep up with the modern
 standards.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.68645ce32e89ff5423c251014f8dcbb8%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+--
 Reporter:  brian  |Owner:  nobody
 Type:  Bug|   Status:  closed
Component:  contrib.admin  |  Version:  1.7
 Severity:  Normal |   Resolution:  duplicate
 Keywords: | Triage Stage:  Unreviewed
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  0  |UI/UX:  0
---+--
Changes (by timgraham):

 * status:  new => closed
 * needs_better_patch:   => 0
 * resolution:   => duplicate
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 The admin isn't designed to work as XHTML, see #19363.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.dff396e20af7aefaf570b362db4e603c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #23908: XHTML breaks admin site

[Django]

#23908: XHTML breaks admin site
---+
 Reporter:  brian  |  Owner:  nobody
 Type:  Bug| Status:  new
Component:  contrib.admin  |Version:  1.7
 Severity:  Normal |   Keywords:
 Triage Stage:  Unreviewed |  Has patch:  0
Easy pickings:  0  |  UI/UX:  0
---+
 By declaring:

 {{{
 DEFAULT_CONTENT_TYPE = "application/xhtml+xml"
 }}}

 The admin site no longer renders correctly.

 I think the problem is that we haven't declared the XML namespace
 correctly, e.g.:

 {{{
 http://www.w3.org/1999/xhtml;>
 }}}

 There are also some other trivial problems, e.g.  no longer works
 (used in breadcrumbs).

 Thanks

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/048.dad129130dfdd27b48d756338127e844%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.