Skip to content

sushi.core.js#1

Open
couto wants to merge 2 commits intoinf0rmer:masterfrom
couto:master
Open

sushi.core.js#1
couto wants to merge 2 commits intoinf0rmer:masterfrom
couto:master

Conversation

@couto
Copy link

@couto couto commented Mar 17, 2011

Ok... Antes de aceitares isto, fica a coisa de que apenas quero mandar o sushi.core.js ate aprender a usar o github como deve ser, peço que seja paciente e vás aguentando com os meus erros...

Acabei a desistir da ideia do Object.create() visto que iria fazer com que cada modulo adicionado tivesse no seu prototype um abuso de metodos ja existes anteriormente... penso que esse overhead é escusado.

No entanto, pessoalmente nao gosto da ideia de a library fazer merge com o objecto Sushi caso ele já exista anteriormente, dai que de momento guarda o Sushi ja existente, e ocupa o seu lugar... adicionei o noConflict para remapear a library para outro nome, e repor o objecto antigo... caso seja necessário.

Não é que tenhas de aceitar obviamente mas é só para veres caso queiras, pessoalmente prefiro o noConflict mas claro, é a tua library =)

Luís Couto added 2 commits March 16, 2011 21:16
	modified:   src/sushi.utils.lang.js
	modified:   test/core.html
	modified:   test/lang.html
@inf0rmer
Copy link
Owner

O merge com um objecto anterior serve para poderes passar configurações antes do init da library. Concordo que at the moment não é muito útil mas pode vir a ser. Anyway, a ideia do noConflict continua válida :). Uma cena: põe a pasta docs/* no gitignore, porque senão não consigo perceber os teus commits com todo o lixo que lá vai :P


Bruno Abrantes
+351 91 711 6358
http://www.brunoabrantes.com

On Mar 17, 2011, at 11:41 , Couto wrote:

Ok... Antes de aceitares isto, fica a coisa de que apenas quero mandar o sushi.core.js ate aprender a usar o github como deve ser, peço que seja paciente e vás aguentando com os meus erros...

Acabei a desistir da ideia do Object.create() visto que iria fazer com que cada modulo adicionado tivesse no seu prototype um abuso de metodos ja existes anteriormente... penso que esse overhead é escusado.

No entanto, pessoalmente nao gosto da ideia de a library fazer merge com o objecto Sushi caso ele já exista anteriormente, dai que de momento guarda o Sushi ja existente, e ocupa o seu lugar... adicionei o noConflict para remapear a library para outro nome, e repor o objecto antigo... caso seja necessário.

Não é que tenhas de aceitar obviamente mas é só para veres caso queiras, pessoalmente prefiro o noConflict mas claro, é a tua library =)

Reply to this email directly or view it on GitHub:
#1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants