Yes, it works perfectly. Thanks

Not really, that's already taken care of in Falkon code: https://cgit.kde.org/falkon.git/tree/src/lib/plugins/ocssupport.cpp - it's detected by the contents of the archive.

With themes it's just a matter of extracting the archive to ~/.config/falkon/themes. Falkon will register the new theme without restart.

Would it be possible to enable the "ocs://" links for both Falkon categories?
I have implemented the url handler in Falkon, so it will only work when installing from Falkon itself, but in case of web browser I don't think it makes much sense to install the addons with external tools.

Full name + email please so I can commit it for you.

Actually ...

Please also to stable branch.

One unexpected thing is that it will also focus search line when opening new browser window, not just when manually opening sidebar.

Looks good.

Alright, please tell me your full name + email and I'll commit part of this patch and adjust the rest myself.

I like it, thanks!

This is wrong, there two ways to add button to notification that opens file/directory:

Take a look in src/plugins src/scripts

I meant that I don't want this feature. Context menu on page is already very long, and if it contained every action that someone requested on bugzilla, it would not even fit on screen.

As with similar actions, this should be in plugin (preferably only script) and not in main code.

Looks good now!

I don't really like that this port became full of workarounds to be honest, but if it works and there's no other way then fine.

In D15304#398048, @ngraham wrote:

Thanks for the approval, but the scrollviews still don't actually scroll when the window is too small to fit all the content! I haven't been able to figure out why this is yet, so help would be appreciated if you have the time. :)