New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link Nofollow #198

Open
tiggeymone opened this Issue Sep 20, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@tiggeymone

tiggeymone commented Sep 20, 2017

Salve, mi chiedevo se fosse prevista l'aggiunta della possibilità di inserire il rel="nofollow" nell'anchor del link per per la pagina della cookie policy.

Attualmente ho provato a modificare manualmente i diversi .js la funzione _createInformationLink:

function _createInformationLink(linkText, linkHref) {
var buttonStyle='color: '+ btcB +';padding: 7px 12px;font-size: 18px;line-height: 18px;text-decoration: none;text-transform: uppercase;margin-right: 20px;margin-bottom:2px;letter-spacing: 0.125em;' +
'display: inline-block;font-weight: normal; text-align: center; vertical-align: middle; cursor: pointer; border: 1px solid '+ btcB +';background: rgba(255, 255, 255, 0.03);';

  var infoLink = document.createElement('a');
  _setElementText(infoLink, linkText);
  infoLink.className = infoClass;
  infoLink.href = linkHref;
  • infoLink.rel = 'nofollow'
    if (tar) infoLink.target = '_blank';
    infoLink.style.cssText = buttonStyle;
    return infoLink;
    }

In locale questa modifica mi permette di ottenere il link nofollow, mentre in remoto non funziona, anche cancellando tutta la cache. Qualche idea? Grazie in anticipo.

@tiggeymone

This comment has been minimized.

tiggeymone commented Sep 20, 2017

Ok, ho capito che il plugin carica i js solo se WP_DEBUG è attivo, altrimenti carica dinamicamente il JS da php. Ho modificato il js/php minified e sono riuscito nel mio scopo. Resta comunque interessante la possibilità di aggiungere il nofollow come opzione nel pannello di amministrazione.

Grazie per l'ottimo lavoro!

@overclokk

This comment has been minimized.

Contributor

overclokk commented Oct 2, 2017

Ciao @tiggeymone
me le segno da aggiungere un una futura release appena riesco ad avere un po' di tempo :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment