[dev] Code review VisualTag infantil

[Catalin Francu]

>
> E adevărat și ce zici tu, dar uite un avantaj al folosirii jCropului:
> Voluntarul încadrează (cu selecția) la fix textul și coordonatele centrului
> sunt determinate automat, indiferent dacă imaginea a fost scalată sau nu.
> Nu cred că toți graficienii o să facă toate imaginile la aceeași
> dimensiune. Unele pot fi mai late, altele mai înalte, în funcție de ce este
> reprezentat acolo. Astfel putem face divul care încadrează imaginea să aibă
> o dimensiune maximă fixă, să nu ocupe tot ecranul și în același timp
> obținem coordonatele de pe imagine la dimensiunea originală.


A, da, asta precis este util, dar atunci înțeleg că definirea unui tag se
face în doi pași: (1) definirea textului și a coordonatelor și (2)
selectarea textului pentru calcularea centrului. Pasul (2) riscă să aibă
erori mărișoare (cel puțin cu mouse-ul meu...). Ideal (ignorând faptul că
unele din aceste operații sunt grele în Javascript) aș vrea să dau click în
poză și să tastez text, atât.

> Etichetele input le-am închis conform [1]. Dar am să le închid cu />
> de-acum încolo.
>
Aha! Nu știam, uite, am învățat ceva nou. Deci doar în XHTML / XML trebuie
<br />.

> Și aș avea o rugăminte: îmi poți spune în ce fișiere trebuie să fac
> modificări pentru a adăuga dreptul PRIV_VISUAL?
>
Cred că trebuie să faci aceste lucruri:

- în phplib/util.php, adaugă o linie nouă PRIV_VISUAL după toate celelalte
constante PRIV_*
- nu uita să incrementezi NUM_PRIVILEGES și să extinzi PRIV_NAMES cu o
valoare rezonabilă
- poți găsi modele de folosire pentru aceste valori cu un mare grep, să
zicem

grep -r PRIV_SUPER *|grep -v templates_c | grep -v svn
grep -r NUM_PRIVILEGES *|grep -v templates_c | grep -v svn

- dacă nu ai făcut deja asta, ar trebui să-ți dai drept de moderator pe
instalarea ta locală. În MySQL:

update User set moderator = 63 where email = '...';

- apoi poți să încarci pagina /moderatori și ar trebui să vezi noua
coloană. Salvează câteva valori și asigură-te că se propagă!

Fă un commit doar cu codul ăsta, dacă poți, ca să lucrăm modular.

Spor,
Cătălin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.dexonline.ro/pipermail/dev/attachments/20130801/b0960bbf/attachment.html>