Can you check whether this affects Task manager also?

In D10835#213787, @broulik wrote:

Can you check whether this affects Task manager also?

Task Manager runs the corresponding application explicitly by its .desktop file, so it's not affected.

So why not do the same here instead of offering "Open With"?

In D10835#214197, @broulik wrote:

So why not do the same here instead of offering "Open With"?

Currently recent documents in application launchers (i.e. Kicker, Kickoff etc.) are opened in their default applications rather than the ones they were opened with back then. My intent was to prevent accidental execution, not to change the behavior.

If you think that current behavior should be changed so that recent documents are opened in the same applications, I'll implement it (in a new revision).

Stupid me, you're absolutely right. Looks good to me but please let @hein have the final word on this.

It looks good to me as well. I was waiting for Kai's concerns to get sorted out ;)

@aleksejshilin Do you have a contributor account to commit this or should we do it on your behalf? (Your authorship will be preserved, of course.)

In D10835#214865, @hein wrote:

Do you have a contributor account to commit this or should we do it on your behalf?

I have a contributor account, thanks.

I'll update the revision to address code style issues pointed out by @anthonyfieroni and will be ready to land it when it's accepted.

If there are no more concerns, could anyone actually accept the revision, please? :) Thanks in advance!