D14137: CSS: update syntax and fix some errors

2018-07-17 Thread Dominik Haumann
dhaumann added a comment. Thanks for pushing. Could you have a quick look at https://build.kde.org and search for "syntax-highlighting"? It seems the unit tests do not pass since this change, probably a minor issue: https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%

D14137: CSS: update syntax and fix some errors

2018-07-16 Thread jonathan poelen
jpoelen added a comment. It is done. So much emotion :') REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D14137 To: jpoelen, dhaumann Cc: kwrite-devel, kde-frameworks-devel, michaelh, genethomas, kevinapavew, ngraham, bruns, demsking, cullmann, vkrause, s

D14137: CSS: update syntax and fix some errors

2018-07-16 Thread jonathan poelen
This revision was automatically updated to reflect the committed changes. Closed by commit R216:423239c45ec5: CSS: update syntax and fix some errors (authored by jpoelen). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D14137?vs=37815&id=37912 REVI

D14137: CSS: update syntax and fix some errors

2018-07-15 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Looks like a nice update - thanks for your work. Can you 'arc land' yourself? :-) REPOSITORY R216 Syntax Highlighting BRANCH css REVISION DETAIL https://phabricator.kde.org/D141

D14137: CSS: update syntax and fix some errors

2018-07-15 Thread jonathan poelen
jpoelen created this revision. jpoelen added a reviewer: dhaumann. jpoelen added projects: Frameworks, Framework: Syntax Highlighting. Restricted Application added a project: Kate. Restricted Application edited subscribers, added: kde-frameworks-devel, kwrite-devel; removed: Frameworks. jpoelen re