[dev] [commit] r992 - in wwwbase: . admin img/colorbox img/lightbox js styles

[Cătălin Frâncu]

Arată bine în general!

> +++ wwwbase/js/dexGallery.js	Sat Sep 21 00:21:06 2013	(r992)

> +  $('canvas').css('position', 'absolute').css('top', '5px').css('left', '5px')
> +    .attr('width', img.css('width')).attr('height', img.css('height'));
> +}

E vreo șansă să facem chestiile astea într-un fișier CSS?

Poate ar fi mai bine să creezi elementul canvas în fișierul .ihtml și să-l ții cu display:none până când ai nevoie de el.

> +function removeCanvas() {
> +  var canvasElement = document.getElementsByTagName('canvas')[0];
> +
> +  $(canvasElement).clearCanvas();
> +  canvasElement.parentNode.removeChild(canvasElement);
> +}

Ar fi mai bine să selectezi canvas-ul după ID, cum ar fi $('#canvasId'). Cu [0] cum e acum, presupune că nu există alt canvas înaintea lui. Cine știe ce ne rezervă viitorul. :-)

> +  canvas.drawText({
> +    layer: true,
> +    name: 'pre' + tagNo,
> +    groups: ['Pre'],
> +    fromCenter: true,
> +    strokeStyle: '#fff',
> +    strokeWidth: 2,
> +    fontSize: 12,
> +    fontFamily: 'Calibri',
> +    text: tagData[4],
> +    maxWidth: tagNameMaxWidth,
> +    x: tagData[0], y: tagData[1],
> +  })

Cât de universal e Calibri ăsta? Ar fi bine să pui un backup bine suportat, Arial sau ceva.

> +  /* Desenează un dreptungi de dimensiunea textului + un delta */

Super, ce mișto că se poate!

> +
> +  /* Rescrie textul în pătrat */

Aici parcă e un pic de duplicare. Bănuiesc că asta e necesar deoarece dreptunghiul suprascrie textul? Nu poți să faci dreptunghiul de culoarea transparentă (adică numai borderul să fie vizibil)? Sau asta ar face să se vadă linia de sub el?

> +    click: function() {
> +      window.open('http://www.dexonline.ro/definitie/' + tagData[4], '_self');
> +    }

E rezonabil așa. Cred că ar fi și mai mișto să poți pune cumva elemente <a> cu aceste linkuri, pentru ca motoarele de căutare să învețe legăturile astea. Dar mi se pare că nu poți pune HTML în canvas.

Auzi, și mai am o meta-întrebare: chestia asta e HTML5? E ok dacă da, dar atunci am să vorbesc cu Sonia să urgentăm migrarea la HTML5 (search.php e HTML4 în prezent).

Cătălin