D8451: JavaScript: include standard mime types

2017-11-08 Thread Nikita Skovoroda 
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:1d55f0448725: JavaScript: include standard mime types 
(authored by ChALkeR).

CHANGED PRIOR TO COMMIT
  https://phabricator.kde.org/D8451?vs=21248&id=22085#toc

REPOSITORY
  R216 Syntax Highlighting

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8451?vs=21248&id=22085

REVISION DETAIL
  https://phabricator.kde.org/D8451

AFFECTED FILES
  data/syntax/javascript.xml

To: ChALkeR, #framework_syntax_highlighting, vkrause
Cc: vkrause, #frameworks

D8451: JavaScript: include standard mime types

2017-10-26 Thread Volker Krause 
vkrause accepted this revision.
vkrause added a comment.
This revision is now accepted and ready to land.


  please also bump the version number

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D8451

To: ChALkeR, #framework_syntax_highlighting, vkrause
Cc: vkrause, #frameworks

D8451: JavaScript: include standard mime types

2017-10-25 Thread Nikita Skovoroda 
ChALkeR edited the summary of this revision.

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D8451

To: ChALkeR, #framework_syntax_highlighting
Cc: #frameworks

D8451: JavaScript: include standard mime types

2017-10-24 Thread Nikita Skovoroda 
ChALkeR edited the summary of this revision.
ChALkeR added a reviewer: Framework: Syntax Highlighting.

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D8451

To: ChALkeR, #framework_syntax_highlighting
Cc: #frameworks

D8451: JavaScript: include standard mime types

2017-10-24 Thread Nikita Skovoroda 
ChALkeR created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.

REVISION SUMMARY
  Add application/javascript and text/javascript mimetypes to
  javascript.xml
  
  application/javascript is the main one used in shared-mime-info,
  without it syntax highlighting is not working for *.mjs files.
  
  text/javascript is added as a safeguard in case if shared-mime-info
  switches to it later, as there is ambiguity between which of those two
  should be the main one, and IANA could switch to text/javascript.
  
  Refs:
  
  - 
https://cgit.freedesktop.org/xdg/shared-mime-info/tree/freedesktop.org.xml.in
  - https://www.iana.org/assignments/media-types/application/javascript
  - https://tools.ietf.org/html/rfc4329
  - https://html.spec.whatwg.org/multipage/scripting.html#javascript-mime-type
  - https://github.com/bmeck/I-D/blob/master/javascript-mjs/draft.md

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D8451

AFFECTED FILES
  data/syntax/javascript.xml

To: ChALkeR
Cc: #frameworks