Re: Ask about new KDE functionnalities.

FWIW: http://www.alysbrooks.com/gpt-4-is-a-risky-dependency-for-foss-projects.html (found at https://ervin.ipsquad.net/blog/2023/04/21/web-review-week-2023-16/). signature.asc Description: PGP signature

Re: Can we rename gitklient?

On Thu, Mar 02, 2023 at 10:58:18PM +0300, Gleb Popov wrote: > I really like "Kommit". It starts with "K" and is just to the point. FWIW - I do not intend to discourage you from renaming the program to "Kommit" - there is a Git (and Mercurial) commit message editor for GNOME called "Commit" (and

Re: build_clazy_clang_tidy job can't find python-yaml

Docker runs the test using a Debian testing‐based image, with both the testing and the unstable repositories enabled. apt-get fails to install python-yaml, because it has been removed from both of these repositories. The Python 3 equivalent is python3-yaml. signature.asc Description: PGP

D29084: Make the HTML file template more useful

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R241:5adaa7cba102: Make the HTML file template more useful (authored by gszymaszek). REPOSITORY R241 KIO CHANGES SINCE

D29084: Make the HTML file template more useful

gszymaszek requested review of this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D29084 To: gszymaszek, #frameworks, dfaure, ognarb Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29084: Make the HTML file template more useful

gszymaszek updated this revision to Diff 80859. gszymaszek added a comment. - Add common HTML elements to the HTML file template REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29084?vs=80853=80859 BRANCH html-file-template (branched from master) REVISION

D29084: Make the HTML file template more useful

gszymaszek updated this revision to Diff 80853. gszymaszek added a comment. - Remove stylesheets from the HTML file template REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29084?vs=80839=80853 BRANCH html-file-template (branched from master) REVISION

D29084: Make the HTML file template more useful

gszymaszek added a comment. In D29084#654484 , @ognarb wrote: > - Don't include

D29084: Make the HTML file template more useful

gszymaszek created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. gszymaszek requested review of this revision. REVISION SUMMARY This commit removes the empty meta tag and adds the following: - a DOCTYPE declaration, - an html[lang]

Re: Polish translation of "Cancel"

dows. Again, I feel you are still not convinced that _most_ people see this translation as “absurd” etc. New and existing users will complain, they will waste hours reading and writing new #404286 comments, blog posts, emails, patches, until you revert the translation. How long are you going to wait?

Re: Polish translation of "Cancel"

On Wed, Apr 15, 2020 at 09:30:34AM -0600, Nate Graham wrote: > Shouldn't the mailing list be using standard KDE infrastructure? Shouldn’t the Polish translation use common vocabulary instead of reviving words that no one uses in real life? I’m afraid both answers are “it should”. -- Grzegorz

Re: Polish translation of "Cancel"

Hi, On Wed, Apr 15, 2020 at 08:45:17AM -0600, Nate Graham wrote: > [+ kde-devel mailing list since I don't know if I got the right localization > mailing list and couldn't find a polish-specific one] FWIW, there are three mailing lists hosted at SourceForge:

D24306: Highlight GeoJSON files as if they were plain JSON

This revision was automatically updated to reflect the committed changes. Closed by commit R216:25b08fe3f4c6: Highlight GeoJSON files as if they were plain JSON (authored by gszymaszek). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE

D24306: Highlight GeoJSON files as if they were plain JSON

gszymaszek created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. gszymaszek requested review of this revision. REVISION SUMMARY Currently, GeoJSON files syntax isn’t highlighted in any way. GeoJSON is a subset of JSON

D18148: Use HTTPS, if available, in links visible to users

This revision was automatically updated to reflect the committed changes. Closed by commit R39:b64681408d84: Use HTTPS, if available, in links visible to users (authored by gszymaszek). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18148?vs=49138=49140

D18148: Use HTTPS, if available, in links visible to users

gszymaszek marked 2 inline comments as done. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18148 To: gszymaszek, #ktexteditor Cc: lueck, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann

D18148: Use HTTPS, if available, in links visible to users

gszymaszek updated this revision to Diff 49138. gszymaszek added a comment. Fixed two links to docs.kde.org REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D18148?vs=49135=49138 BRANCH https-in-end-user-links (branched from master) REVISION DETAIL

D18148: Use HTTPS, if available, in links visible to users

gszymaszek retitled this revision from "Use HTTPS, if available, in links visible by users" to "Use HTTPS, if available, in links visible to users". gszymaszek added a reviewer: KTextEditor. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D18148 To: gszymaszek,

D18148: Use HTTPS, if available, in links visible by users

gszymaszek created this revision. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. gszymaszek requested review of this revision. REPOSITORY R39 KTextEditor BRANCH https-in-end-user-links (branched from master) REVISION DETAIL

D7864: Basic support for PHP and Python fenced code blocks in Markdown

gszymaszek updated this revision to Diff 19631. gszymaszek added a comment. - Removed JavaScript code blocks in Markdown support since it didn’t work - Added test cases for new code blocks in Markdown REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE

D7864: Basic support for PHP and Python fenced code blocks in Markdown

gszymaszek added a subscriber: cfeck. gszymaszek added a comment. Not ready to land, but uploaded per @cfeck’s suggestion (bug 356974 ). REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D7864 To: gszymaszek,

D7864: Basic support for PHP and Python fenced code blocks in Markdown

gszymaszek created this revision. gszymaszek added a reviewer: KTextEditor. Restricted Application added a project: Frameworks. REVISION SUMMARY This patch enables syntax highlighting of PHP and Python code blocks in Markdown code. REPOSITORY R216 Syntax Highlighting BRANCH

D5799: Rebase Less syntax highlighting on SCSS one

This revision was automatically updated to reflect the committed changes. Closed by commit R216:ca846fbda318: Rebased Less syntax highlighting on SCSS one (authored by gszymaszek). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5799?vs=18296=18391

D5799: Rebase Less syntax highlighting on SCSS one

gszymaszek updated this revision to Diff 18296. gszymaszek added a comment. Removed leading and trailing spaces inside XML tags (in `less.xml`), to make it possible to build the framework. REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE

Re: Need HTML/CSS help for Konqueror's about page

Hello, It seems Chromium interprets the following (lines 23–24): like this: In HTML, a div must have both opening and closing tags, so you should change the original code to: to make it render correctly. You will face the same problem in lines 101 and 104. Other issues: 1.

D5799: Rebase Less syntax highlighting on SCSS one

gszymaszek updated this revision to Diff 15814. gszymaszek added a comment. Added a test file (`autotests/input/highlight.less`). REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5799?vs=14376=15814 REVISION DETAIL

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. In https://phabricator.kde.org/D4537#87411, @cullmann wrote: > Do you have commit rights? If not, paste your mail address and full name and I can push your change with you as author. No, I don’t have. Grzegorz Szymaszek, `gszymas...@short

[Differential] [Updated, 256 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11486. gszymaszek marked 5 inline comments as done. gszymaszek added a comment. Updated to work with new `cmake/FindEditorConfig.cmake`. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4537?vs=11366=11486

[Differential] [Updated, 259 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11366. gszymaszek added a comment. key and value are const. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4537?vs=11365=11366 REVISION DETAIL https://phabricator.kde.org/D4537 AFFECTED FILES CMakeLists.txt

[Differential] [Updated, 259 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11365. gszymaszek marked 2 inline comments as done. gszymaszek added a comment. Pulled check-variable-functions out of EditorConfig class, moved some variable definitions into main loop of parser. REPOSITORY R39 KTextEditor CHANGES SINCE LAST

[Differential] [Updated, 254 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11349. gszymaszek added a comment. Simplified the constructor and reduced parser indentation. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4537?vs=11279=11349 REVISION DETAIL

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. Yes, it was good when actual parsing was delegated to a separate function, but we’ve decided to merge all parsing-related functions. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D4537 EMAIL PREFERENCES

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. In https://phabricator.kde.org/D4537#86356, @cullmann wrote: > On the other side, I see no real copyrightable material beside you are using the editor config API. As I’ve written, I used Builder’s source to //learn// that (1) I have to call

[Differential] [Updated, 88 lines] D4589: EditorConfig module

gszymaszek updated this revision to Diff 11329. gszymaszek added a comment. Added description. REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4589?vs=11308=11329 REVISION DETAIL https://phabricator.kde.org/D4589 AFFECTED FILES

[Differential] [Updated, 67 lines] D4589: EditorConfig module

gszymaszek updated this revision to Diff 11308. gszymaszek added a comment. Added version checking support. REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4589?vs=11263=11308 REVISION DETAIL https://phabricator.kde.org/D4589 AFFECTED FILES

[Differential] [Commented On] D4589: EditorConfig module

gszymaszek added a comment. editorconfig-core-lua doesn’t check C library version (CMakeLists.txt:49 ). Actually I’m not sure if setting oldest

[Differential] [Commented On] D4589: EditorConfig module

gszymaszek added a comment. It seems there isn’t any version string in library headers (editorconfig.h, editorconfig_handle.h), is there any other cross-distribution way to obtain library version? REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D4589

[Differential] [Updated, 253 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11279. gszymaszek added a comment. Fixed src/CMakeLists.txt to make EditorConfig support truly optional, removed interpret and interpretLine functions (so parsing is back in parse), simplified EditorConfig class constructor, fixed memory leak in

[Differential] [Updated] D4537: EditorConfig support

gszymaszek marked 6 inline comments as done. gszymaszek added inline comments. INLINE COMMENTS > dhaumann wrote in editorconfig.cpp:23 > Better is to write: > > EditorConfig::EditorConfig(KTextEditor::DocumentPrivate *document) > : m_document(document) > , m_handle(0) > { >

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. In https://phabricator.kde.org/D4537#85846, @dhaumann wrote: > Btw, did you copy some code from another project? If so, we need to be careful, since KTextEditor is LGPLv+2. I’ve used mentioned gnome-builder’s file

[Differential] [Request, 41 lines] D4589: EditorConfig module

gszymaszek created this revision. gszymaszek added reviewers: Build System, Frameworks. gszymaszek set the repository for this revision to R240 Extra CMake Modules. gszymaszek added a project: Frameworks. Restricted Application added a project: Build System. REVISION SUMMARY EditorConfig

[Differential] [Updated, 265 lines] D4537: EditorConfig support

gszymaszek updated this revision to Diff 11262. gszymaszek added a comment. Moved EditorConfig-related logic into a separate class and made it optional (no longer a KTextEditor dependency). REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. In the new `EditorConfig` class I need to use `DocumentPrivate`’s `checkBoolValue` and `checkIntValue` methods, but they’re declared private. I think it should be OK to make them public as they’re static anyway, but I’d like to hear what’s your opinion. Setting

[Differential] [Commented On] D4537: EditorConfig support

gszymaszek added a comment. Thanks for your comments. I’ve managed to make editorconfig optional using the mentioned `FindEditorConfig.cmake` module (I had to change its name to `Findeditorconfig.cmake`). Should I create a new diff in ECM repository to add this file? Now I’m going to

[Differential] [Request, 109 lines] D4537: EditorConfig support

gszymaszek created this revision. gszymaszek added a reviewer: KTextEditor. gszymaszek set the repository for this revision to R39 KTextEditor. Restricted Application added subscribers: Frameworks, kwrite-devel. Restricted Application added a project: Frameworks. REVISION SUMMARY This diff

[Differential] [Request, 2 lines] D4300: Simpler calculation of longest line limit

gszymaszek created this revision. gszymaszek added a reviewer: KTextEditor. gszymaszek set the repository for this revision to R39 KTextEditor. gszymaszek added a project: KTextEditor. Restricted Application added subscribers: Frameworks, kwrite-devel. Restricted Application added a project:

[Differential] [Request, 4 lines] D4296: Fixed single line comments in Less starting new regions

gszymaszek created this revision. gszymaszek added a reviewer: Framework: Syntax Hightlighting. gszymaszek set the repository for this revision to R216 Syntax Highlighting. gszymaszek added a project: Framework: Syntax Hightlighting. Restricted Application added a project: Frameworks. Restricted