dhaumann (Dominik Haumann)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Apr 20 2015, 7:20 AM (252 w, 2 d)
Availability
Available

Recent Activity

Thu, Feb 13

dhaumann updated subscribers of D27285: Add left/right indent fill (as opposed to left-only), extend indent lines to broken lines.

Ok, I now fully got this, thanks for the explanation. And indeed without any left fill visual feedback is missing.

Thu, Feb 13, 10:21 PM · Frameworks, Kate

Wed, Feb 12

dhaumann added a comment to D27285: Add left/right indent fill (as opposed to left-only), extend indent lines to broken lines.

No offense meant, but even with the screenshots I still have no idea what this is about :)

Wed, Feb 12, 11:04 PM · Frameworks, Kate

Sun, Feb 9

dhaumann updated subscribers of D27238: Add an option to dynamic-break inside words.

I think this patch looks good.

Sun, Feb 9, 9:43 PM · Frameworks, KTextEditor, Kate

Thu, Feb 6

dhaumann added a comment to D27180: Add sortable columns to session manager dialog, and remove previous sorting code.

I'd be fine with committing this. Still, there very clearly is large room to improve this further.

Thu, Feb 6, 11:01 PM · Kate
dhaumann added a comment to D27180: Add sortable columns to session manager dialog, and remove previous sorting code.

I like the idea of the patch, could you add a screenshot?

Thu, Feb 6, 9:31 PM · Kate

Thu, Jan 30

dhaumann added a comment to D27035: [KMessageWidget] Draw it with QPainter instead of using stylesheet.

Better :) with corners I mean the 1-3 pixels left due to the rounding corners. These pixels were once also drawn as background although they are outside of the frame. It may be a minor detail, but imho such details are important. But indeed, the screenshots look good.

Thu, Jan 30, 9:19 PM · Frameworks
dhaumann added a comment to D26697: Message-related methods: Use more member-function-pointer-based connect.

In two days we have the next KDE Frameworks tag. I'd have preferred a commit on Sunday ;)

Thu, Jan 30, 9:05 PM · Frameworks, Kate
dhaumann added a comment to D27035: [KMessageWidget] Draw it with QPainter instead of using stylesheet.

Can you provide screenshots of more styles? :-)

Thu, Jan 30, 4:30 PM · Frameworks
dhaumann added a comment to D27035: [KMessageWidget] Draw it with QPainter instead of using stylesheet.

Does that also work in Kate for floating messages like when the search wraps? What I want to know is whether the corners behind the green frame are transparent in this case, or whether the corners are painted solid. If I remember correctly, these kind of bugs were the reason to use Qt StyleSheets. And it must work with all styles.

Thu, Jan 30, 4:19 PM · Frameworks

Fri, Jan 24

dhaumann added a comment to D26903: allow programLogo property to be a QIcon, too.

Looks good to me, and very much in line with the other canConvert statements before.

Fri, Jan 24, 8:43 PM · Frameworks

Tue, Jan 21

dhaumann committed R40:d198c37cdb36: Fix External Tool "Google Selected Text" (authored by dhaumann).
Fix External Tool "Google Selected Text"
Tue, Jan 21, 6:07 PM
dhaumann committed R40:a07dceabbb02: Fix External Tool "Google Selected Text" (authored by dhaumann).
Fix External Tool "Google Selected Text"
Tue, Jan 21, 6:02 PM
dhaumann closed D26812: Variable expansion: Add variable PercentEncoded.
Tue, Jan 21, 5:33 PM · Frameworks, Kate
dhaumann committed R39:b0055e8476bb: Variable expansion: Add variable PercentEncoded (authored by dhaumann).
Variable expansion: Add variable PercentEncoded
Tue, Jan 21, 5:33 PM
dhaumann committed R39:22fa707eb7e2: Fix crash in variable expansion (used by external tools) (authored by dhaumann).
Fix crash in variable expansion (used by external tools)
Tue, Jan 21, 5:33 PM
dhaumann closed D26811: Fix crash in variable expansion (used by external tools).
Tue, Jan 21, 5:33 PM · Frameworks, Kate
dhaumann requested review of D26812: Variable expansion: Add variable PercentEncoded.
Tue, Jan 21, 5:31 PM · Frameworks, Kate
dhaumann requested review of D26811: Fix crash in variable expansion (used by external tools).
Tue, Jan 21, 5:13 PM · Frameworks, Kate
dhaumann committed R216:e8e098bd9f28: Perl: Add 'say' to keyword list (authored by dhaumann).
Perl: Add 'say' to keyword list
Tue, Jan 21, 11:20 AM
dhaumann added inline comments to D25698: New query mechanism for applications: KApplicationTrader.
Tue, Jan 21, 7:37 AM · Frameworks

Jan 20 2020

dhaumann accepted D26779: Updates nasm.xml with the latest instructions.

Looks good to me. Can you commit or shall we push this for you?

Jan 20 2020, 2:01 PM · Frameworks, Kate

Jan 19 2020

dhaumann added a comment to D26755: [WIP] KMessageWidget: Set widget height on resize event.

Well, it definitely would be very nice if you find a patch that fixes this.

Jan 19 2020, 2:53 PM · Frameworks
dhaumann resigned from D26755: [WIP] KMessageWidget: Set widget height on resize event.

In that case I withdraw my review for now. I can only speak for Kate that I could not find a regression with my tests. But if you find other regressions, then this is likely not correct.

Jan 19 2020, 12:28 PM · Frameworks
dhaumann committed R39:d47bf03792fc: GIT_SILENT: Adapt timings of message test (authored by dhaumann).
GIT_SILENT: Adapt timings of message test
Jan 19 2020, 11:41 AM
dhaumann accepted D26755: [WIP] KMessageWidget: Set widget height on resize event.

I just tested with KTextEditor's messagetest and KTextEditor's usage of KMessageWidget in the top and bottom bar, and all this looks still good.

Jan 19 2020, 11:34 AM · Frameworks
dhaumann committed R40:631f6a3e4e7f: QUrl: Use isLocalFile() to test for valid local file (authored by dhaumann).
QUrl: Use isLocalFile() to test for valid local file
Jan 19 2020, 11:06 AM
dhaumann committed R40:df2b580f1d6f: External tools: Correctly set the working directory (authored by dhaumann).
External tools: Correctly set the working directory
Jan 19 2020, 11:00 AM
dhaumann added a comment to D26755: [WIP] KMessageWidget: Set widget height on resize event.

Since KTextEditor uses KMessageWidget extensively we have to make sure the unit test messagetest still passes with this change.

Jan 19 2020, 8:46 AM · Frameworks

Jan 16 2020

dhaumann accepted D26704: KateMessageWidget: remove unused event filter installation.

I guess this change is correct :-)

Jan 16 2020, 3:48 PM · Frameworks, Kate

Jan 13 2020

dhaumann added a comment to D26560: Fix selection printing.

That looks already better: If a line is wrapped and on the same page, then only the selected text is printed. It seems there are still corner cases.

  1. Create a document with just one long line that wraps over two printed pages. I this case, I am not able to print only the selected text properly.
  2. Say you have a line that wraps over e.g. 5 visual lines. And you just want to print the visually wrapped line 4 and 5. Right now, the visual lines 1-3 are just empty, but still take place. Maybe it makes sense to omit fully empty lines completely?
Jan 13 2020, 7:51 PM · Frameworks, Kate

Jan 10 2020

dhaumann committed R40:007a7aef9999: Highlight documentation: add equivalence in regex for number rules and add… (authored by nibags).
Highlight documentation: add equivalence in regex for number rules and add…
Jan 10 2020, 8:09 PM

Jan 6 2020

dhaumann accepted D26463: Port endl as \n or std::cout ... std::endl when we wrote directly on console or flush QTextStream when it's a qstring.
Jan 6 2020, 8:19 PM · Frameworks, Kate

Jan 5 2020

dhaumann added inline comments to D26424: [kdiroperator] Add method for accessing actions without KActionCollection.
Jan 5 2020, 9:45 AM · Frameworks

Jan 3 2020

dhaumann committed R40:adb1a3564abb: Remove Qt 5.7 version check, since Kate depends on Qt 5.10 (authored by dhaumann).
Remove Qt 5.7 version check, since Kate depends on Qt 5.10
Jan 3 2020, 2:28 PM
dhaumann committed R40:b451a73f6055: Remove QT_VERSION_CHECK for Qt >= 5.7, since Kate depends on Qt 5.10 (authored by dhaumann).
Remove QT_VERSION_CHECK for Qt >= 5.7, since Kate depends on Qt 5.10
Jan 3 2020, 2:14 PM
dhaumann committed R39:6609e4f9235d: Fix warning about unused variable and minor optimization (authored by dhaumann).
Fix warning about unused variable and minor optimization
Jan 3 2020, 2:13 PM
dhaumann committed R39:1520fced9ccf: GIT_SILENT: Adapt qDebug() output to proper spaces and quotes (authored by dhaumann).
GIT_SILENT: Adapt qDebug() output to proper spaces and quotes
Jan 3 2020, 2:13 PM
dhaumann committed R40:6b5f1ee7a3e2: Merge branch 'release/19.12' (authored by dhaumann).
Merge branch 'release/19.12'
Jan 3 2020, 1:47 PM
dhaumann committed R40:5aa3b009cf53: Fix crash when pressing ESC after external tools plugin was un/reloaded (authored by dhaumann).
Fix crash when pressing ESC after external tools plugin was un/reloaded
Jan 3 2020, 1:46 PM

Jan 1 2020

dhaumann accepted D26321: Expose show-line-count in the ConfigInterface.

Ok with this... Thanks. A note " Since 5.6x would be nice".

Jan 1 2020, 12:38 AM · Frameworks, KTextEditor, Kate

Dec 30 2019

dhaumann committed R39:46ff95a99746: Branch for KSyntaxHighlighting::Theme integration (authored by dhaumann).
Branch for KSyntaxHighlighting::Theme integration
Dec 30 2019, 10:03 AM
dhaumann committed R40:5203a17cc497: Merge branch 'release/19.12' (authored by dhaumann).
Merge branch 'release/19.12'
Dec 30 2019, 9:54 AM

Dec 29 2019

dhaumann added a comment to D21563: Fix empty pages in print preview and lines printed twice.

@hoffmannrobert: by the way, looking at your phabricator activity, you should get a KDE contributor account, if you don't have one already.

Dec 29 2019, 11:06 AM · Frameworks, Kate
dhaumann added a comment to D21563: Fix empty pages in print preview and lines printed twice.

@hoffmannrobert: Are you maybe also interested in looking into https://bugs.kde.org/show_bug.cgi?id=415570 ? It's again about printing, this time about the very last line that seems to be wrong...

Dec 29 2019, 11:03 AM · Frameworks, Kate

Dec 22 2019

dhaumann added a comment to T12318: Make Dolphin's and Kate's terminal shortcuts and behaviour consistent.

@Gregor: That the terminal does not get focus on show was a regression in konsole part and should be fixed. I don't think there is any action item left for now.

Dec 22 2019, 10:16 PM · Goal: Consistency, Dolphin, Kate

Dec 15 2019

dhaumann committed R40:5d486c09a859: doc: extend lspclient documentation (authored by mnauwelaerts).
doc: extend lspclient documentation
Dec 15 2019, 10:59 PM

Dec 11 2019

dhaumann accepted D25870: SELinux: add "glblub" keyword and update permissions list.
Dec 11 2019, 2:07 PM · Frameworks, Kate
dhaumann accepted D25869: Python: improve numbers, add octals, binaries and "breakpoint" keyword.
Dec 11 2019, 2:06 PM · Frameworks, Kate

Dec 10 2019

dhaumann added a comment to D23457: Port the search interface from QRegExp to QRegularExpression.

Personally, I am not convinced replacing \s with [ \t], and deviating from PCRE default behaviour is a good idea in this case.

Dec 10 2019, 9:11 PM · Frameworks, Kate
dhaumann added a comment to D23457: Port the search interface from QRegExp to QRegularExpression.

Was anything of the previously commented issues addressed?

Dec 10 2019, 1:43 PM · Frameworks, Kate

Dec 8 2019

dhaumann added a comment to D25805: Open/close and focus/defocus terminal using the same shortcuts as Dolphin.

Doesn't this patch imply we have the same action twice now? Once provided by Konsole, and once by katemdi via View > Tool Views > Show Terminal?

Dec 8 2019, 10:56 PM · Kate

Dec 6 2019

dhaumann added inline comments to D25767: KAutoSaveFile: add a unit test to check max. filename length.
Dec 6 2019, 8:13 AM · Frameworks

Dec 5 2019

dhaumann added a comment to D25755: Replace iterators with range-based for.

Do we dislike iterators now?

We don't, and they still make sense for when you need the key, but range for is just much nier to look at :)

I'm fine with that statement. But are we going to be reviewing changing all the KDE code from iterators to range for? Feels like an overkill to me.

Dec 5 2019, 8:18 PM · Frameworks
dhaumann accepted D25770: Several enhancements to gitolite syntax definition..

Looks good. But does the highlighting work for RW+CD? I am wondering whether + needs to be added to the weakDeliminator list?

Dec 5 2019, 8:06 PM · Frameworks, Kate

Dec 4 2019

dhaumann added a comment to T12318: Make Dolphin's and Kate's terminal shortcuts and behaviour consistent.

You can achieve Dolphin's behavior by reassigning the F4 shortcut to View > Tool Views > Show/Hide Konsole. That's what I do since > 10 years, and it works perfectly. Why is it not the default? Maybe because this action does not exist by default (i.e. when the Konsole plugin is not loaded). But the right fix is to reassign the F4 shortcut to this action.

Dec 4 2019, 3:50 PM · Goal: Consistency, Dolphin, Kate

Dec 3 2019

dhaumann accepted D25276: reStructuredText: Fix inline literals highlighting preceding characters.

Ok

Dec 3 2019, 6:59 AM · Frameworks, Kate

Dec 2 2019

dhaumann added inline comments to D25698: New query mechanism for applications: KApplicationTrader.
Dec 2 2019, 9:55 PM · Frameworks
dhaumann committed R40:4003304a4d13: Merge branch 'fix-translation' into release/19.12 (authored by dhaumann).
Merge branch 'fix-translation' into release/19.12
Dec 2 2019, 9:15 PM
dhaumann committed R40:f65192bbe6a6: Fix translation of external tools (authored by dhaumann).
Fix translation of external tools
Dec 2 2019, 8:46 PM
dhaumann committed R40:df460443bf6e: Fix: Do not translate KActionCollection identifier (authored by dhaumann).
Fix: Do not translate KActionCollection identifier
Dec 2 2019, 8:35 PM

Nov 26 2019

dhaumann added a comment to D25493: EBN kmoretools cleanup transport protocol.

@jhayes Do you have commit access, or shall I commit this for you?

Nov 26 2019, 10:47 PM · Frameworks
dhaumann added a comment to D23457: Port the search interface from QRegExp to QRegularExpression.

Any news here?

Nov 26 2019, 10:45 PM · Frameworks, Kate
dhaumann added a comment to D24982: Small improvements in some XML files.

I think we should decide what to do with this patch, as over time it will get merge conflicts.

Nov 26 2019, 10:42 PM · Frameworks, Kate

Nov 24 2019

dhaumann committed R304:c8fa3ab044cb: Make KF6 cleanup work visible (authored by dhaumann).
Make KF6 cleanup work visible
Nov 24 2019, 6:03 PM
dhaumann closed D25508: Make KF6 cleanup work visible.
Nov 24 2019, 6:03 PM · Frameworks
dhaumann renamed T12244: Move enum KSyntaxHighlighting::TextStyles from KSyntaxHighlighting Text Styles to Move enum KSyntaxHighlighting::TextStyles.
Nov 24 2019, 5:57 PM · KF6
dhaumann created T12244: Move enum KSyntaxHighlighting::TextStyles.
Nov 24 2019, 5:56 PM · KF6
dhaumann created T12243: KSyntaxHighlighting: Expose context in highlighting functions.
Nov 24 2019, 5:41 PM · KF6
dhaumann requested review of D25508: Make KF6 cleanup work visible.
Nov 24 2019, 5:12 PM · Frameworks
dhaumann added a comment to D25503: JavaScript: move keywords from TypeScript and other improvements.

I think this is ok.

Nov 24 2019, 2:37 PM · Frameworks, Kate
dhaumann accepted D25493: EBN kmoretools cleanup transport protocol.
Nov 24 2019, 7:03 AM · Frameworks
dhaumann added a comment to D17595: Upstream Dolphin's file rename dialog.

I received the agreements of all contributors including the original code author, to relicense this code and hence upstream it.

Nov 24 2019, 7:01 AM · Frameworks

Nov 23 2019

dhaumann added a comment to D25485: Remove KIconThemes use.

Makes sense, thanks.

Nov 23 2019, 9:51 AM · Kate, Frameworks

Nov 22 2019

dhaumann accepted D25465: [DropJob] Minor grammar fix in API docs.

Lgtm.

Nov 22 2019, 4:48 PM · Frameworks

Nov 19 2019

dhaumann added inline comments to D25276: reStructuredText: Fix inline literals highlighting preceding characters.
Nov 19 2019, 8:44 PM · Frameworks, Kate

Nov 16 2019

dhaumann added a comment to D25328: Always rehiglhight() after definition was changed.

Can't you call rehighlight() yourself after calling setDefinition()?

Nov 16 2019, 6:09 PM · Frameworks, Kate
dhaumann added a comment to D24568: Provide clang-format target with a KDE Frameworks style file.

You can force the current clang format to keep the multi-line if as follows:

Nov 16 2019, 8:03 AM · Build System, Frameworks

Nov 11 2019

dhaumann committed R40:b99cceb1f7f4: Merge branch 'add-json-xml-formatters' into 'master' (authored by dhaumann).
Merge branch 'add-json-xml-formatters' into 'master'
Nov 11 2019, 8:56 PM

Nov 10 2019

dhaumann updated subscribers of T11864: Integrate KQuickSyntaxHighlighter into KSyntaxHighlighting.

@hein: Could you add a commit that relicenses all code to MIT? We switched to MIT for entire KSyntaxHighlighting. This will ease integration later.

Nov 10 2019, 9:10 PM · KF6
dhaumann accepted D25232: kate: port away from deprecated methods in KWindowSystem >= 5.62.

Thanks, please commit! PS: Kate is also on gitlab: When pushing your branch to KDE's kate.git, the git push output will tell you how to create the merge request. But you can push directly as well, if you want.

Nov 10 2019, 2:18 PM · Kate

Oct 31 2019

dhaumann added a comment to T11535: Organize 2019 KF6 sprint.

Also can't make it.

Oct 31 2019, 6:59 PM · KF6

Oct 29 2019

dhaumann added a comment to D25054: Add string escape characters to PowerShell syntax..

Looks good to me.

Oct 29 2019, 9:34 PM · Frameworks, Kate
dhaumann accepted D25054: Add string escape characters to PowerShell syntax..

Please commit.

Oct 29 2019, 6:54 PM · Frameworks, Kate

Oct 28 2019

dhaumann added a comment to D24982: Small improvements in some XML files.

Hm right... too bad. I was hoping to find an automated way to detect this. Since relying on the user to optimize the RegExps will always be suboptimal. @cullmann Do you have any ideas?

Oct 28 2019, 9:31 PM · Frameworks, Kate
dhaumann added a comment to D24983: KateModeMenuList: improve word wrap.

Good, fine with me.

Oct 28 2019, 3:38 PM · Frameworks, Kate
dhaumann added a comment to D24982: Small improvements in some XML files.

[...]
One option would be to add a capture or dontCapture attribute to enable or disable captures for RegExpr rules. Also, captures could be enabled or disabled in all RegExpr rules using the <general> group, adding an element for that.

Oct 28 2019, 3:34 PM · Frameworks, Kate

Oct 27 2019

dhaumann accepted D24983: KateModeMenuList: improve word wrap.

It's hard to review all the pixel changes with no screenshots. It's even unclear to me what problem exactly is fixed at hand. Given it's all your code, I trust you know what you are doing...

Oct 27 2019, 4:45 PM · Frameworks, Kate
dhaumann added a comment to D24982: Small improvements in some XML files.

I wonder if the ?: optimizations make sense. QRegularExpression has the option QRegularExpression::DontCaptureOption to not capture anything. Looking into our code we have:

Oct 27 2019, 4:32 PM · Frameworks, Kate
dhaumann accepted D24981: Modelines: fix end of comment.
Oct 27 2019, 1:26 PM · Frameworks, Kate

Oct 25 2019

dhaumann accepted D24939: Meson: more built-in functions and add built-in member functions.

Looks good to me, thanks!

Oct 25 2019, 5:43 AM · Frameworks, Kate

Oct 20 2019

dhaumann accepted D24620: Windows MSVC compile fix.

Imo we should simply try: We have another two weeks for testing.

Oct 20 2019, 7:32 PM · Frameworks

Oct 19 2019

dhaumann committed R40:1fbfc2a37879: Merge branch 'update-docs' into 'master' (authored by dhaumann).
Merge branch 'update-docs' into 'master'
Oct 19 2019, 8:20 PM
dhaumann committed R40:b520021b1771: Update documentation of highlight & RegExp (authored by nibags).
Update documentation of highlight & RegExp
Oct 19 2019, 8:19 PM

Oct 12 2019

dhaumann added a comment to T9585: Publish Okteta in the Microsoft Store.

@kossebau Just tried Okteta on Windows that is available on https://binary-factory.kde.org/job/Okteta_Nightly_win64/ and it works really well, out of the box. I believe it really has a lot of value making Okteta more visible, especially since users on Windows would also use it on Linux.

Oct 12 2019, 7:23 PM · Windows
dhaumann added a comment to D24568: Provide clang-format target with a KDE Frameworks style file.

I'm all for it. This would unify how we can reformat any KDE module, which is very much desirable.

Oct 12 2019, 7:30 AM · Build System, Frameworks
dhaumann accepted D24578: Meson: Add a comment section for comment/uncomment with Kate.

Please increase the version number and commit.

Oct 12 2019, 7:12 AM · Frameworks, Kate
dhaumann accepted D24575: Updates from CMake 3.16.

Thanks for the quick update.

Oct 12 2019, 7:10 AM · Frameworks, Kate

Oct 10 2019

dhaumann moved T10319: Ideas for External Tools from Backlog to Done on the Kate board.

The External Tools plugin is done for now. I would like to wait a bit to get feedback, and over time we can certainly add more default tools, or even extend the plugin with additional functionality.

Oct 10 2019, 8:38 AM · Kate
dhaumann moved T10281: Create an "Outline View" from Backlog to Done on the Kate board.

I believe work should be invested into the LSP client, since it provides an outline as well. Over time, the language servers will get better and better, so you'll get all this for free.

Oct 10 2019, 8:37 AM · Kate
dhaumann moved T10249: Revive External Tools Plugin from In Progress to Done on the Kate board.
Oct 10 2019, 8:34 AM · Kate