D14828: remove highlighting download dialog

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


  Independent of mom/dad/somebody else, for highlighting files you create on 
your own, you just can perfectly add them to the local directories as specified 
in the manual.
  If you really want to have new highlighting files and risk that stuff like 
indenters or included stuff might break, you can still get them from the 
syntax-highlighting git and put them in that folders manually, too.
  The only thing that is no longer there is the "auto" download dialog.

REPOSITORY
  R39 KTextEditor

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

To: cullmann, dhaumann
Cc: rjvbb, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D14828: remove highlighting download dialog

2018-08-14 Thread René J . V . Bertin
rjvbb added a comment.


  Downloading already didn't work for me since quite a while (the operation 
never completed) but what does this mean for highlighting files I write myself 
(or someone's mom does)?
  
  > Other users who do not follow the latest development releases [...] are 
likely not the target audience for downloading highlighting files.
  
  I take a bit exception with this; users of certain distributions that aim to 
allow them to work *with* the software instead of *on* (as in applying updates) 
are being compared with a rather negative and probably sexist view of their 
mother. The opposite is true for many such users: working with stable releases 
means you have more reason to update things like highlighting files.
  
  FWIW, there was never a reason to have a download/update button in the 
"userland" configure UI; a KCM provided by the highlighting framework would 
make perfect sense (that same KCM could serve to disable/uninstall useless 
highlighting files; your mom probably doesn't need more than half the ones that 
get installed.

REPOSITORY
  R39 KTextEditor

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

To: cullmann, dhaumann
Cc: rjvbb, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, 
bruns, demsking, cullmann, sars, dhaumann


D14828: remove highlighting download dialog

2018-08-14 Thread Christoph Cullmann
This revision was automatically updated to reflect the committed changes.
Closed by commit R39:705f8e0419c4: remove highlighting download dialog 
(authored by cullmann).

REPOSITORY
  R39 KTextEditor

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D14828?vs=39686=39688

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

AFFECTED FILES
  src/dialogs/filetypeconfigwidget.ui
  src/dialogs/katedialogs.cpp
  src/dialogs/katedialogs.h
  src/mode/katemodeconfigpage.cpp
  src/mode/katemodeconfigpage.h

To: cullmann, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, 
demsking, cullmann, sars, dhaumann


D14828: remove highlighting download dialog

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


  Looks good. But along with this change, please change the docbook in kate.git:
  
  We have doc/katepart/configuring.docbook
  

Download Highlighting Files...
  

  Click this button to download new or updated syntax highlight 
descriptions
  from the  website.

  

  
  And we have: doc/katepart/development.docbook
  

  You can download updated or additional syntax highlight definitions 
from the  website by clicking the
  Download Highlighting Files... button in the Modes  Filetypes tab of 
the Config Dialog.


REPOSITORY
  R39 KTextEditor

BRANCH
  no-hl-download (branched from master)

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

To: cullmann, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, 
demsking, cullmann, sars, dhaumann


D14828: remove highlighting download dialog

2018-08-14 Thread Christoph Cullmann
cullmann created this revision.
cullmann added a reviewer: dhaumann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
cullmann requested review of this revision.

REVISION SUMMARY
  rational:
  
  1. there is one frameworks release per month that already provides a stream 
of hl updates
  2. often new highlightings anyways need a new frameworks version
  3. one can easily update only a part of the files which leads to broken other 
highlighting that include parts of them
  4. the updates might easily break the non-downloadable indenters
  
  GUI: highlighting download dialog removed

TEST PLAN
  make
  make test

REPOSITORY
  R39 KTextEditor

BRANCH
  no-hl-download (branched from master)

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

AFFECTED FILES
  src/dialogs/filetypeconfigwidget.ui
  src/dialogs/katedialogs.cpp
  src/dialogs/katedialogs.h
  src/mode/katemodeconfigpage.cpp
  src/mode/katemodeconfigpage.h

To: cullmann, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, 
demsking, cullmann, sars, dhaumann