Modify ↓
Opened 17 years ago
Closed 17 years ago
#5392 closed defect (fixed)
Xsl, Xslt syntax highlighting
Reported by: | damien | Owned by: | Emmanuel Blot |
---|---|---|---|
Priority: | normal | Milestone: | 0.11 |
Component: | version control/browser | Version: | devel |
Severity: | normal | Keywords: | xsl xslt |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
source:trunk/trac/mimeview/api.py@5388#L99
This line should be changed from
'text/xml': ['xml'],
to
'text/xml': ['xml','xsl','xslt'],
and line 100 should be removed since pygments handles xsl as 'text/xml'. This will fix xsl syntax highlighting, (I have tested it).
Attachments (0)
Change History (3)
comment:1 by , 17 years ago
comment:2 by , 17 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 17 years ago
Milestone: | 0.11.1 → 0.11 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Implemented in [5901]
Note:
See TracTickets
for help on using tickets.
Replying to damien:
I was a bit reluctant at first to drop
text/xsl
MIME type in favor oftext/xml
for XSL file. However, it appears thattext/xsl
has never been officially registered as a MIME type, it is mostly an IE thing.However, there is another alternative for XSL files:
application/xsl+xml
. This MIME type works also with Pygments, and carries more information astext/xml
does.OTOH,
api.py
should not be defined for Pygments, as Pygments is only one choice in the mime converter galaxy.See also RFC 3023, Appendix A: about the motivation for using
+xml
suffix.As a side note, the following MIME types are recognized by Pygments as supported XML types:
text/xml
application/xml
image/svg+xml
application/rss+xml
application/atom+xml
application/xsl+xml
application/xslt+xml