Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

oprava lessfiltru, preklep, pridan dokumentace #41

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

pvy commented Apr 11, 2013

Ahoj,

  • prvni commit je ukazka pouziti lessfilteru, protoze jsem to musel hledat a nejdrive jsem si ani nevsiml, ze se rozlisuje filters a fileFilters
  • druhy je doplnena chybejici zavorka
  • treti si nejsem jist, zda je nutny: teoreticky umoznuje aplikovat less i na spojeny obsah. Ostatni filtry to maji. Samozřejmě by to takhle nic nedělalo, ale ani nepadalo.
  • ctvrty commit umoznuje injectovat vlastni lessC. Diky tomu se da prizpuzobit (odkud includovat, zda includovat z aktualniho adresare, co s komentarema,...). Mel by byt zpetne kompatibilni, protoze bez dalsi konfigurace postaru vytvari novou instanci a protoze ta ma implicitne nastaven importDir na "", tak i importuje ze stejneho mista, jako je filtrovany soubor.

tuhle část moc nechápu

Owner

pvy replied Jun 5, 2013

Prázdný řetězec znamená import z aktuálního adresáře (=stejného, jako právě zpracovávaný soubor).
Takže si ho dočasně přidám a po zpracování zase zahodím, aby při zpracování jiného souboru z jiného adresáře nedošlo ke kolizi nebo neoprávněnému načtení.

Experimentoval jsem s nastavením, kdy webloader hledá všechny less soubory v adresáři komponenty (někde uvnitř app/components), které se porůznu importovaly zase jen v rámci té komponenty a měly být izolované od zbytku světa.

(už je to nějakou dobu, tak snad jsem to zpětně dovodil správně ;))

tohle mi nepřijde jak moc dobrej nápad. spíš přemýšlím, že by měl less filter nebo css urls filter uživatele seřvat, pokud jej zkusí zaregistrovat jako filter a ne jako fileFilter

Owner

pvy replied Jun 5, 2013

To je také možnost, ale u StylusFilter.php to je také volitelné a předpokládám, že je to také jen pro filefilter (nepoužívám, nevím jistě).

Pokud je to pravda, chtělo by to pro transparentnost sjednotit.

@fprochazka fprochazka commented on the diff Nov 23, 2013

WebLoader/Filter/LessFilter.php
* @license MIT
*/
class LessFilter
{
- private $lc;
+ private $lessC;
+
+ /**
+ * Makes possible to inject a customized object
+ * @param \lessc $lessC
+ */
+ public function setLessC(\lessc $lessC){
@fprochazka

fprochazka Nov 23, 2013

Collaborator

Závorka patří na další řádek.

Collaborator

fprochazka commented Nov 24, 2013

Upravil jsem to na 0ce1214 a můžeš nyní předat instanci lessc přes konstruktor, což je čistější.

@fprochazka fprochazka closed this Nov 24, 2013

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