help-librejs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Help-librejs] [Bug-librejs] Supporting <link rel="jslicense">


From: Sebastian Noack
Subject: Re: [Help-librejs] [Bug-librejs] Supporting <link rel="jslicense">
Date: Sat, 22 Feb 2014 17:39:28 +0100

Hi Loic,

you might be able to override a "display: none;" style, but then websites that want to hide the link, can still just position it out of the visible area, remove it with _javascript_ after the site was loaded, or use the Shadow DOM to hide it. So if somebody wants to hide the web labels link, he can do it without that LibreJS will be able to know that it is hidden.

Beside the technical difficulties to prevent hiding the web labels link, I think there are legit cases, where we should allow websites to hide that link. For example on pages that doesn't contain any other navigation elements, like pages that show media in full screen.

So wouldn't it be better to allow websites to hide the link without any hacks, with a <link> tag in the header. And show the link, doesn't matter whether it is visible on the the web page, additionally in the LibreJS overlay?

Cheers
Sebastian


On Sat, Feb 22, 2014 at 5:08 PM, Loic J. Duros <address@hidden> wrote:

Actually, I'm going to run this by the boss, since I'm not sure we want
to encourage web developers to hide the link. One thing LibreJS could do
is force the link to be displayed. This way anyone with LibreJS would
actually see the link, even if the CSS on the page attempts to hide it.

Sebastian Noack <address@hidden> writes:

> Hi,
>
> a lot of websites (including https://www.eff.org/) hide the web labels link
> anyway with CSS. So why don't let people use a link-tag as an alternative to
> an a-tag to specify their web labels page, in case they don't want have a
> visible link? We can't force them to show the link anyway, but hiding it with
> CSS looks like a hack, since that is what the link-tag is for. You find the
> patch in the attachment.
>
> Cheers
> Sebastian
>
>
> diff -Naur /tmp/librejs-5.4.1.old/lib/html_script_finder/web_labels/js_web_labels.js /tmp/librejs-5.4.1.new/lib/html_script_finder/web_labels/js_web_labels.js
> --- /tmp/librejs-5.4.1.old/lib/html_script_finder/web_labels/js_web_labels.js 2013-10-26 00:35:13.000000000 +0200
> +++ /tmp/librejs-5.4.1.new/lib/html_script_finder/web_labels/js_web_labels.js 2014-02-04 22:43:58.723030048 +0100
> @@ -86,7 +86,7 @@
>
>    searchForJsLink: function () {
>         //console.log('triggered searchForJsLink');
> -       var linkTags = this.dom.getElementsByTagName('a'),
> +       var linkTags = this.dom.querySelectorAll('a,link'),
>             i = 0,
>             len = linkTags.length,
>             path;


reply via email to

[Prev in Thread] Current Thread [Next in Thread]