There are some minor TODOs here and there still left. I will try addressing some of those this week.

Thanks! I might have already done more work in that direction if there wasn't this task with many comments pushing towards removing contextual help in general

The UI, yes, to the extent possible. Which is often not the case because there is simply no place for additional text (if that's what you were suggesting) and even if there were it would lead to a lot of visual clutter.

I think a solution like https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/51 would be a good fit for the Kate/KTextEditor config dialog.

I just stumbled upon this bug (issue #1) recently and it is quite annoying. I believe this is a bug in Kate, and not in Qt and it triggers with the space character. So, if you are writing a line which hits the border and wraps, it wont have this issue unless the last character before the wrap was a space.

In D28406#644383, @dfaure wrote:

@waqar ping? What do you think about my suggestion? If you agree, can you update the patch?

In D28406#637359, @dfaure wrote:

I'm no expert in this stuff, but here's my feedback

1. on my own system (!= CI), all three tests were failing before, due to finding en_AU-variant_1, en_AU-variant_1 and "".
2. now the test passes locally, but because all tests are skipped SKIP : SonnetAutoDetectTest::autodetect(English) en_US-large not available SKIP : SonnetAutoDetectTest::autodetect(German) de_DE_frami not available SKIP : SonnetAutoDetectTest::autodetect(Malayam) ml_IN not available What's en_US-large? Can we at least check that we get en_US-*?

Please have a look and submit fixes.

I tried but I don't have the access.

In D25495#633551, @apol wrote:

Whatever, let's go with this and we can learn along the way.

Sorry it took so long.

In D25495#608582, @apol wrote:

This patch makes it do all the languages identified by identify + all the languages supported by the script. This could mean lots of languages for some scripts...

ping

Apply requested changes

Apply requested changes

Hi,
First of all thanks for reviewing.

ping

Yes, I think so too.

With no respect for your lack of being subtle

Add test for iso code to language names

Sorry but i don't think this is an acceptable solution,

In D25530#567472, @aacid wrote:

Any reason we particularly need this here instead of improving Qt?

Remove accent from Росси́я

Remove unused includes

Update copyright

Add Test for autodetection

Add test for autodetection

Revert a change, as autodetection fails for languages with similar scripts

Fix Sonnet autodetect completely failing when no trigrams found