<p dir="ltr">Probabil astă seară am să trimit un code review complet. Am făcut întocmai cum mi-ai spus, am făcut o clonă nouă, curată și acolo am început să modific. Un svn diff pe vechea clonă ar fi fost urât :) acum sigur o să fie mai simpatic. </p>
<p dir="ltr">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ă.</p>
<p dir="ltr">Etichetele input le-am închis conform [1]. Dar am să le închid cu /> de-acum încolo.</p>
<p dir="ltr">Îmi pare rău pentru codul împrăștiat, dar mi-a fost puțin mai ușor așa. Am să fac ordine. Sper să reușesc să fac astă seară. Deocamdată sunt în drum spre București. La noapte am să plec la mare. Problema este că ne obligă să schimbăm căminul și dacă nu o să trebuiască să începem mutările din seara asta, o să mă ocup de observații.</p>
<p dir="ltr">Ș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?</p>
<p dir="ltr">[1] <a href="http://www.w3schools.com/tags/tag_input.asp">http://www.w3schools.com/tags/tag_input.asp</a></p>
<p dir="ltr">Alex</p>
<div class="gmail_quote">On 31 Jul 2013 12:39, "Catalin Francu" <<a href="mailto:cata@francu.com">cata@francu.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
[Cc: dev]<div><br></div><div>Ok, acum am înțeles! Atunci mă bucur cu atât mai mult că te-ai prins cum să folosești jCrop. Rămâne de văzut dacă îl vom folosi în interfața finală, căci s-ar putea să nu mai avem nevoie să decupăm dreptunghiuri. De fapt, lungimea textului determină lățimea dreptunghiului.</div>
<div><br></div><div>Din păcate, nu pot să rulez codul de la tine, căci îmi lipsește visualTag.php. De-asta e mai simplu să-mi trimiți un diff. :-) Dar iată niște observații:</div><div><br></div><div>- visualTag.ihtml: stilurile ar trebui separate într-un fișier css, fie într-un fișier nou, visualTag.css, dacă va fi mult, fie în flex.css. Flex.css este un amalgam cu tot CSS-ul din paginile de admin și am să-l restructurez eu într-o bună zi cu soare.</div>
<div><br></div><div>- Pentru textele vizibile de utilizator, hai să folosim „etichetă” și „etichetare”. Sună mai românește. :-) Știu că e ciudat să scriem cod cu keywords în engleză și mesaje în română, dar există o separație destul de clară.</div>
<div><br></div><div>- Etichetele <input ... /> se închid cu /> (ca și toate care nu vin în pereche, ca <button>...</button></div><div><br></div><div>- Nu îmi este clar unde stă modsVisualTag.php. Bănuiesc că tot în admin? Practica noastră este să punem toată logica în visualTag.php, să nu facem un fișier separat pentru salvare. Iar visualTag va începe fix cu aceste if-uri pe care le ai tu, după care va continua cu codul în cazul în care nu există parametri la salvare (pagina este încărcată prima oară).</div>
<div><br></div><div>- Dacă ai în continuare nevoie de redirectare, folosește util_redirect(). E un pic mai deșteaptă, în special pentru că propagă mesajele de eroare pentru utilizator.</div><div><br></div><div>În rest, fă-i commit. Abia aștept să mă joc cu interfața!</div>
<div><br></div><div>Cătălin<br><br><div class="gmail_quote">2013/7/30 Grigoroiu Marian Alexandru <span dir="ltr"><<a href="mailto:grigoroiualexandru@gmail.com" target="_blank">grigoroiualexandru@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Probabil nu am fost destul de explicit. Coordonatele nu trebuie introduse de către utilizator. Sub imagine am afișat coordonatele pentru a vedea care sunt și pentru a mă ajuta pe mine la debugging. <div>
<br>
</div><div><img src="cid:ii_1403018a55d93eb0" alt="Imagini inline 1" width="671" height="383"><br></div><div>Ele se actualizează automat când coordonatele selecției se modifică. Când utilizatorul e mulțumit de selecție, apasă pe butonul Setează Coordonatele și sunt preluate automat și completate câmpurile. </div>
<div><br></div><div><img src="cid:ii_140301aa26640a56" alt="Imagini inline 2" width="671" height="383"><br></div><div>Poate acum am reușit să fiu mai explicit cu imaginile. Pot încerca să fac și ceea ce zici tu, asta pentru un fine-tunning, dacă voluntarului nu i se pare ok partea cu selecția cu mouse-ul, sau la o eventuală editare a tagului.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">În data de 30 iulie 2013, 17:56, Catalin Francu <span dir="ltr"><<a href="mailto:cata@francu.com" target="_blank">cata@francu.com</a>></span> a scris:<div>
<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Salutare,<div><br></div><div>E un început foarte bun! </div><div><br></div><div>Aș vrea să trecem, ușor-ușor, spre o interfață mai ușor de folosit cu mouse-ul. Anticipez că oamenii vor găsi greoaie o interfață unde trebuie să introducă coordonate. În cel mai rău caz, putem folosi <a href="http://jqueryui.com/spinner/" target="_blank">spinner</a> de la jQueryUI ca să facem o chestie hibridă, mouse + săgeți.</div>
<div><br></div><div>Dar ce ziceți de designul ăsta: sub imagine sunt afișate tag-urile, unul pe rând, și fiecare are două seturi de câte patru săgeți pentru mișcarea poziției etichetei și a vârfului săgeții. Ceva cam așa (ASCII art):</div>
<div><br></div><div><font face="courier new, monospace"> ^ ^</font></div><div><font face="courier new, monospace">cornee < 223,455 > < 357,622 > [x] șterge</font></div><div>
<font face="courier new, monospace"> v v</font></div><div><font face="courier new, monospace"><br></font></div><div><div><font face="courier new, monospace"> ^ ^</font></div>
<div><font face="courier new, monospace">cristalin < 223,455 > < 357,622 > [x] șterge</font></div><div><font face="courier new, monospace"> v v</font></div></div><div><br></div>
<div>Ideea e că nici nu vreau să ne omorâm cu Javascript monstruos, dar nici să nu oferim o interfață prea spartană.</div><div><br></div><div>Dacă pare prea complex, hai să ne vedem când puteți și tu și Radu și să o batem în cuie. La mine calendarul e lejer de vineri încolo.</div>
<div><br></div><div>Cătălin<br><br><div class="gmail_quote"><div><div>2013/7/29 Grigoroiu Marian Alexandru <span dir="ltr"><<a href="mailto:grigoroiualexandru@gmail.com" target="_blank">grigoroiualexandru@gmail.com</a>></span><br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Am făcut o interfață nu pe deplin funcțională a tăguirii. Ea arată cam așa:<div>
<br></div><div>Se încarcă prima imagine din tabelul Visual care are 0 în coloana revised.</div>
<div><br></div><div><img src="cid:ii_140270915f1255d2" alt="Imagini inline 1" width="474" height="271"></div>
<div>Id-ul imaginii se încarcă automat. Sub imagine se actualizează coordonatele centrului selecției. Cu un click pe oricare dintre butoanele „setează” sunt copiate coordonatele în câmpurile alăturate. Odată salvate, tagurile vor apărea sub taguri salvate, cu opțiunea de a fi șterse.</div>
<div><br></div><div><img src="cid:ii_140270e3e9143e95" alt="Imagini inline 2" width="474" height="270"></div><div>Mă gândesc și la opțiunea de editare. Odată cu click pe butonul Tăguirea este completă, imaginii i se va modifica la 1 valoarea din coloana revised și se va încărca următoarea imagine. Dacă toate imaginile au fost revizuite, va apărea mesajul:</div>
<div><br></div><div><img src="cid:ii_140271338c253988" alt="Imagini inline 3" width="474" height="270"></div><div><div><br></div><div>Cam atât deocamdată :D</div><span><font color="#888888"><div><br></div>
-- <br><div dir="ltr"><div><span style="font-family:georgia,serif">Toate cele bune,</span><br>
</div><div><font face="georgia, serif">Grigoroiu Marian Alexandru</font></div></div>
</font></span></div></div>
<br></div></div>_______________________________________________<br>
Dev mailing list<br>
<a href="mailto:Dev@dexonline.ro" target="_blank">Dev@dexonline.ro</a><br>
<a href="http://list.dexonline.ro/listinfo/dev" target="_blank">http://list.dexonline.ro/listinfo/dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div></div></div><div><div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><span style="font-family:georgia,serif">Toate cele bune,</span><br></div><div><font face="georgia, serif">Grigoroiu Marian Alexandru</font></div>
</div>
</div></div></div>
</blockquote></div><br></div>
</blockquote></div>