Re: [hackers] [surf-webkit2] Add Notification support

From: Charles Lehner <cel_AT_celehner.com>
Date: Wed, 20 Jan 2016 12:06:09 -0500

On Wed, 20 Jan 2016 11:31:11 +0100
quinq <quinq_AT_fifth.space> wrote:

> Hi Charles,
>
> Thank you for your interest in surf-webkit2.
> It is in (more or less) heavy development at this point and the status
> of the suckless repo is kind of frozen.
> I'm working on expanding it a bit in two principal branches outside.
> One for handling communication between the UI and the Rendering
> processes via IPC (fifo between the client and a webextension), another
> for handling parameters as a global object and making it easier to use
> per-domain configuration.
> The third one is a merge of those two.
> You can test those on git.fifth.space/surf which I'll push back to
> suckless when it's more stable.

I see. I've tried out surf-webkit2-full briefly now and it seems to work
well. One thing I found is that in surf-webkit2 (both repos), some fonts
are incorrectly sized. This could just be the site I am using; I have
not fully investigated it, but I only noticed this in surf-webkit2, not
in epiphany (another browser that uses libwebkit2gtk-4.0).

> I'll keep your patch and apply it at some point, thanks!

Great.

Also, a note about the patch: it seems that because of the way
notifications permissions work, toggling notifications to disable
them on a site will not disable them if the site already requested
the permission and surf granted it. Geolocation might be the same
way - I did not check. This might mean that if a site is abusing the
notifications feature (although I have not come across such a site), you
can't simply press the key to disable the notifications - you have to
relaunch surf and disable notifications before visiting the site or
before it requests the permission.
Received on Wed Jan 20 2016 - 18:06:09 CET