-
Notifications
You must be signed in to change notification settings - Fork 176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jquery als externe dependency #1
Comments
Wer shariff in eine Site mit jQuery einbaut, kann das ja entsprechend anpassen. jQuery wird nunmal benötigt, d.h. bei Seiten die noch keins haben müsste es manuell eingebunden werden. Manuell entfernen ist da einfacher. Und externes Einbinden wiederspräche dem Sinn der lib. |
das waere ung. so wie wenn jedes jquery plugin noch ne kopie von jquery ins plugin reinrechnen wuerde :) |
Der Meinung von xat bin ich ebenfalls. |
Kann man jQuery nicht auch als "Geschwister"-Dependency deklarieren, sodass für das Projekt jQuery neben shariff installiert wird, wenn man das via Wenn ich das richtig verstehe, würde das zumindest das Problem mit der doppelten jQuery-Installation lösen. Viel interessanter fände ich jedoch bower Support; dazu mach ich gleich mal ein Ticket auf. |
Hi, ein Build ohne enthaltenes jQuery ist eine gute Idee. Werde ich spätestens Montag nachliefern. Bower-Support sollte auch machbar sein. |
Vielleicht wuerde sich das module hier fuer die Umsetzung anbieten: und dann mit {
"browserify-shim":
{
"jquery": "global:jQuery"
}
} |
Wie sieht es mit ZeptoJS aus? Das ist eine deutlich leichtgewichtigere Bibliothek, deren API mit jQuery weitgehend kompatibel ist. |
Hi, jQuery ist ab jetzt nur noch in "shariff.complete.js" enthalten: Somit kann dieses Ticket wohl geschlossen werden. |
Ich denke es macht mehr Sinn jQuery nicht mit in den build zu packen. Das blaeht die lib nur unnoetig auf (200KB unminimiert) und jQuery hat ohnehin fast jede Webseite schon eingebunden. D.h. die meisten haetten durch diese Lib jQuery dann doppelt eingebunden.
The text was updated successfully, but these errors were encountered: