[Web2.0] Scurire la pagina

Mer 20 Dic 2006 11:49:08 CET

On 12/19/06, Marcello Vezzelli <marcello a vezz.it> wrote:
> Andrea Colanicchia ha scritto:
> >     <script language="javascript">
> >         nav = (typeof document.all!=="undefined")?"ie":"moz";
> il check si usa una volta sola... secondo me è inutile usare una variabile.
> Se si usasse 2 o + volte avrebbe + senso.

In effetti era un pezzo ripreso dal codice vecchio, ma secondo me è +
pulito mettere il risultato del test in una variabile booleana da
usare nell'if dopo.

Anche perché questo è un test che si usa spesso e se estendiamo il
codice la variabile verrà usata spesso in altri if, ma è una questione
di gusti, possiamo anche toglierla se vuoi.

> >         show = false
> Show è un duplicato di display='block'.
> Per questo l'ho rimossa dal mio codice.

OK

> >         function doBlur()
> >         {
> [..]
> >
> >         }
> >
> >         function doUnblur()
> >         {
> [..]
> >
> >         }
> è concettualmente corretto usare le astrazioni doBlur e doUnblur.
> Però a questo punto per evitare di fare tante volte getelement, userei
> una variabile globale che contiene l'elemento in questione e un solo
> getelement.

Ci avevo pensato, poi ho pensato anche che questo avrebbe fatto
rimanere sempre la variabile in memoria occupando spazio, ma siccome è
una variabile sola, si può fare.

> >         function resizeBackground()
> >         {
> >             el = document.getElementById('test');
> >
> >             el.style.width = ((document.documentElement.clientWidth)?
> > document.documentElement.clientWidth : document.body.clientWidth) +
> > "px";
> >             el.style.height = ((document.documentElement.clientHeight)?
> > document.documentElement.clientHeight : document.body.clientHeight) +
> > "px";
> io ho mantenuto document.documentElement.clientHeight e basta perché
> funziona correttamente sia su ie6-7 che su mozilla.
> Ho letto qui:
> http://www.howtocreate.co.uk/tutorials/javascript/browserwindow

Qui non ho controllato, nel senso che ho preso un codice postato da un
altro utente, lo abbiamo testato su vari browser e funziona... non so
cosa dirti, effettivamente penso che l'unico browser rimasto fuori dal
test che bisognerebbe testare è opera, se funziona pure li lasciamo la
tua versione.

> > <body id="body" onload="resizeBackground()">
> a cosa server un resize sull'onload? comunque il div è invisibile, e
> quando viene reso visibile viene ridimensionato.

Il fatto che venga ridimensionato quando viene mostrato è una cosa che
ho aggiunto dopo, quindi all'inizio onload serviva per dare una
dimensione iniziale al div e il resize per modificarlo quando la
finestra cambiava.

Ma ora effettivamente non serve +.

> Ho notato che se non si mette width=100% nel div test, su firefox non
> funziona correttamente.

Non c'era già nell'ultimo post?

> Domani proviamo a mettere tutto insieme.

OK.

Vedi tu avevi postato un codice che sembrava togliere le cose messe
dagli altri senza dare spiegazioni, alla prima vista sembrava
solamente che avessi preso il codice di qualche post precedente e ti
fossi perso le ultime aggiunte.

Se avessi spiegato prima... :)

> Guarda nel mio codice, ho aggiunto il popup e spostato il codice per
> togliere il div opaco. Alla fine l'obiettivo era quello di visualizzare
> una sorta di popup oscurando il resto della pagina, giusto?

Si esatto.

Andrea.