D23135: Gettext: fix "previous untranslated string" and other improvements/fixes

2019-08-15 Thread Christoph Cullmann
cullmann added a comment.


  If the highlighting still is OK after removing them, sure, but it looks like 
the language wanted explicitly have different colors for stuff using the same 
default style. Perhaps one could use some of the new default styles in some 
places.
  And I would always update the version, to allow people to update properly.

REPOSITORY
  R216 Syntax Highlighting

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

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D23135: Gettext: fix "previous untranslated string" and other improvements/fixes

2019-08-15 Thread jonathan poelen
jpoelen added a comment.


  If it is better not to have a hard-coded color, wouldn't it be better to 
remove them once and for all? Without necessarily incrementing the version.

REPOSITORY
  R216 Syntax Highlighting

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

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D23135: Gettext: fix "previous untranslated string" and other improvements/fixes

2019-08-15 Thread Christoph Cullmann
cullmann closed this revision.

REPOSITORY
  R216 Syntax Highlighting

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

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D23135: Gettext: fix "previous untranslated string" and other improvements/fixes

2019-08-14 Thread Dominik Haumann
dhaumann accepted this revision.
dhaumann added a comment.
This revision is now accepted and ready to land.


  Thanks. Removing hardcoded colors would also be nice :-)

REPOSITORY
  R216 Syntax Highlighting

BRANCH
  gettext (branched from master)

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

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D23135: Gettext: fix "previous untranslated string" and other improvements/fixes

2019-08-13 Thread jonathan poelen
jpoelen created this revision.
jpoelen added reviewers: Framework: Syntax Highlighting, dhaumann, cullmann.
jpoelen added projects: Kate, Frameworks.
jpoelen requested review of this revision.

REVISION SUMMARY
  - fix #| (previous untranslated string)
  - fix hexadecimal entity
  - fix html tag in `a < b`
  - use  rather than  for auto-completion
  - string without " at the end of the line is an error
  - # followed by something other than " |:.," is an extension

REPOSITORY
  R216 Syntax Highlighting

BRANCH
  gettext (branched from master)

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

AFFECTED FILES
  autotests/folding/highlight.po.fold
  autotests/html/highlight.po.html
  autotests/input/highlight.po
  autotests/reference/highlight.po.ref
  data/syntax/gettext.xml

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, 
bruns, demsking, cullmann, sars, dhaumann