Skip to content
This repository has been archived by the owner on Jun 17, 2020. It is now read-only.

modifiche parziali per data attribute #11

Merged
merged 5 commits into from
Mar 27, 2014

Conversation

incasteve
Copy link

per ora, come vedi, non ho ancora toccato il discorso del javascript che legge i data attribute perchè mi interessa prima fornire il valore corretto.

@matjack1
Copy link
Owner

ok, allora guarda qua: http://code.google.com/p/phamlp/wiki/HamlConventions

dovresti fare qualcosa tipo

div(data-name="Nome")

e poi da JS puoi fare qualcosa tipo:

$('div').data('name')

ti è utile questo aiuto o hai altri dubbi?

@matjack1
Copy link
Owner

grande @pioneerskies! Il .next() era opera mia direi, hai ragione tu comunque!

@alessandro-fazzi
Copy link
Collaborator

RIguardo #11 (comment) ho visto che stefano nel codice ha usato la sintassi ruby-like per gli attributi e per non confonderlo manterrei quella:

div{ :data-name => "Nome" ) invece di div(data-name="Nome")

no? Anche per coerenza nel progetto

@incasteve
Copy link
Author

@matjack1 : sono un coglione io perchè per la fretta non mi sono accorto che ti ho mandato un codice che in realtà non era completo...
la mia intezione era questa:
%ul
- foreach($projects as $project)
- $nome = $project['Titolo Progetto']
%li.progetto{:name => $nome} = $nome

il problema di per se non era il passaggio dei dati(non ci ero ancora arrivato :P ) ma piuttosto fargli leggere la variabile php.

@pioneerskies grazie per il consiglio, più tardi la leggo bene e ti faccio sapere

@matjack1
Copy link
Owner

@pioneerskies io ho sempre usato l'altra sintassi, che tra l'altro preferisco, è più HTML-style

@incasteve quando hai problemi di sintassi di questo tipo ti consiglio di girare un po' per il codice già scritto per vedere come ho fatto io per risolvere lo stesso problema, di modo che lo stile di scrittura del codice resti coerente e così non ti spacchi la testa a capire come fare.
Per esempio in layouts/geopost.html.haml ho fatto così:

%body(class="#{body_classes()}")

@alessandro-fazzi
Copy link
Collaborator

@matjack1 anche io uso sempre la sintassi HTML-like e se devo votare voto per quella in un prgetto WordPress

@matjack1
Copy link
Owner

❤️

@alessandro-fazzi
Copy link
Collaborator

Scusate ragazzi, ma non avevo ricevuto nessuna notifica.

Io ora faccio il merge, dato che è pending da 10 giorni, poi controllo a posteriori un po' di cose e cerco di rispondere a Stefano. Poco ortodosso, ma facciamo un'eccezzione

alessandro-fazzi added a commit that referenced this pull request Mar 27, 2014
modifiche parziali per data attribute
@alessandro-fazzi alessandro-fazzi merged commit 2cc723a into matjack1:master Mar 27, 2014
@alessandro-fazzi
Copy link
Collaborator

ho aggiornato il codice on-line.

@matjack1
Copy link
Owner

anch'io non avevo ricevuto notifiche! @incasteve, la prossima volta mandaci una mail se non vedi reazioni o cita i nostri nickname!

grazie @pioneerskies :*

@incasteve
Copy link
Author

lo farò ;) non volevo tediarvi più di quanto non stessi già facendo. grazie @pioneerskies per il merge :)
nei prossimi giorni potremmo sentirci un attimo (chi può) su hangout per qualche chiarimento?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants