search

flame-org / Modules

Nette modules on the Steroids
http://flame-org.github.io/Modules/
GNU Lesser General Public License v3.0
13 stars 7 forks source link

Makra by se měla předávat na onCompile #13

Closed jirinapravnik closed 10 years ago

jirinapravnik commented 10 years ago

Stávající implementace nedovoluj překrýt makra implementovaná v Nette. Což je docela problém, když chce někdo překrýt aby inputy a labely přidělávali classy pro bootstrap apod. viz tohle: http://forum.nette.org/cs/17822-lze-prekryt-defaultni-makra-v-nette-2-2

Jo tohle funguje jen v nette 2.2, ale podle composer.json nižší nepodporujete, ne?

jirinapravnik commented 10 years ago

Tak koukám, že neprošly testy. Jenže nevím, co je chybně, protože mě tohle bez problémů funguje, či-li to budu muset nechat asi někomu zkušenějšímu:(

jsifalda commented 10 years ago

Přidáno! Děkujeme:) https://github.com/flame-org/Modules/commit/7b77fa70b33e19d2538a226d5130f2c6f145273b

jirinapravnik commented 10 years ago

Já díky!;-)

jirinapravnik commented 9 years ago

@jsifalda tak bohužel nějak mi zase nejde překrývat makra. Pravděpodobně jak jste dělali tu velkou upravu u router (btw skvělou)

TomasVotruba commented 9 years ago

Možná změnout https://github.com/flame-org/Modules/pull/16/files#diff-e312e8dd6f25dbb0b0c208bd9096b021L143 na https://github.com/flame-org/Modules/pull/16/files#diff-e312e8dd6f25dbb0b0c208bd9096b021R113

Testy prošly a překrývání nepoužívám. Zvládneš přidat test a fixnout to?

jirinapravnik commented 9 years ago

Můžu potvrdit, že když to vrátím, tak to funguje. Test bych i přidal, ale bohužel testovat neumím, resp. jen základy...

TomasVotruba commented 9 years ago

Rozumím. Tak prosím pošli PR s opravou.

jsifalda commented 9 years ago

těším se na pull request :)