Skip to content
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

Moved responsive images rewrite to sl_images #65

Merged
merged 12 commits into from May 12, 2013
Merged

Conversation

lowki
Copy link
Contributor

@lowki lowki commented May 12, 2013

Je ne suis pas certain que ce soit une bonne idée de le mettre là, mais je n'ai pas trouvé mieux.
Au passage la méthode prend un paramètre de plus pour spécifier les tailles et points de bascule que l'on souhaite avoir et sert également des images adaptées aux écrans à haute densité de pixels (Retina).
A terme, il faudrait tout intégrer dans ckeditor, ou configurer les points de rupture et tailles d'images dans le template.
PS: Je ne comprends pas pourquoi, mes anciens commit apparaissent toujours dans le pulll request.

lowki added 11 commits May 3, 2013 11:37
Now you can use chained methods.
Like this $db->setQuery()->execute();
Added db connector choice.
Added some slash config params.
Demo datas are now separated and optionnaly installed.
Conflicts:
	core/common/class/db/mysqli/connector.php
	setup/SlSetup.php
Added responsive images support to sl_articles/sla_articles
Fixed attachments display in sl_articles
Added responsive images support to sl_articles/sla_articles
Fixed attachments display in sl_articles
@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Oula ! Effectivement.
Tu as mis à jour ton dépôt avant ?

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Possibilité de m'arranger ça ? car là j'ai un peu de mal à y voir clair ><

@lowki
Copy link
Contributor Author

lowki commented May 12, 2013

Oui j'ai fais un merge juste avant. Je ne sais pas si on peut effacer des
commit. Je vais me renseigner :)

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Sinon au pire je fais un diff quand j'ai un peu de temps, puis tu vois après le merge pour rétablir ton fork à jour.

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

J'ai un problème avec la règle htaccess qui me pète les miniatures lors de l'upload coté back-office.
Aussi, je me demande si cela ne serai pas mieux d'intégrer le paramètre "adapter les images à l'écran" au niveau global (table sl_config).

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Ah et il manque aussi dans le dump le champ responsive_images ;)
Que pense tu de l'intégration global de ce paramètre ?

@lowki
Copy link
Contributor Author

lowki commented May 12, 2013

Ha, pas moi, tu utilises IE?

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Le troll quoi ^^
Non je parle de la ligne permettant de créer le champ responsive_images dans le dump SQL lors de l'installation ;)

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Ah bin si il est là : lowki@834d032#L8L78

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Pour la miniature dans l'ajax_upload, j'ai bien un problème avec le htaccess ;)

@lowki
Copy link
Contributor Author

lowki commented May 12, 2013

J'étais en train de chercher la ligne :)
Je te trollais sur les miniatures qui ne fonctionnent pas dans le back.
Sinon, je pense qu'il faut que ça reste paramétrable pour chaque page/article.
Ha ok +1 pour l'ajaxupload.

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Coté front j'ai aussi des problèmes d'affichages ><
Et un truc bizarre...

wtf

@wakdev
Copy link
Owner

wakdev commented May 12, 2013

Rah, je n'ai plus d'affichage sur les images responsive ><

@lowki
Copy link
Contributor Author

lowki commented May 12, 2013

Bizarre tout ça, je n'ai pas de bug sur le front.

wakdev added a commit that referenced this pull request May 12, 2013
Moved responsive images rewrite to sl_images
@wakdev wakdev merged commit b08d884 into wakdev:master May 12, 2013
@wakdev
Copy link
Owner

wakdev commented May 13, 2013

J'ai un problème avec la règle : RewriteRule (medias/(?:images|attachments)/..(?:jpe?g|gif|png|JPE?G|GIF|PNG))(?:/(\d\w_))?(?:/(\d_\w*))? core/common/class/functions/includes/show_image.php?url=$1&width=$2&height=$3

Ne fonctionne pas si le site n'est pas à la racine.

@lowki
Copy link
Contributor Author

lowki commented May 13, 2013

Ca ne match pas ou ça donne une mauvaise url?

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Mauvaise url, du coup bin... pas d'image ^^

@lowki
Copy link
Contributor Author

lowki commented May 13, 2013

Marche impec chez moi.
Quelle url il te donne?
T'as quoi comme serveur?

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Je vais re-vérifier quand j'aurai un peu de temps, si j'ai pas une merde sur mon serveur.
Ou alors j'ai pas vidé mon cache mdr ^^

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

404

C'est la même :(

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Obligé de rajouter /slash-cms/ devant la règle.
Du coup : /slash-cms/core/common/class/functions/includes/show_image.php?url=$1&width=$2&height=$3

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

De plus forcement, cela pose aussi problème sur les lignes ErrorDocument

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Putain, je suis une buse en htaccess ><
Je trouve pas la solution lol

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Bon sinon il y a la solution du RewriteBase /slash-cms/
Mais cela oblige l'utilisateur à modifier le htaccess, ce qui est pas terrible.

@lowki
Copy link
Contributor Author

lowki commented May 13, 2013

Colle ça die($url); à la ligne 249 de show_image.php et dis moi ce que ça te sort s'il te plait.

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

http://localhost/slash-cms/medias/images/wakdev.jpg/2000
=> ../../../../../medias/images/wakdev.jpg

Avec le RewriteBase /slash-cms/
Sinon c'est 404

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Nan, c'est vraiment la règle de ré-écriture qui merde. (voir image sans RewriteBase /slash-cms/)
404

@lowki
Copy link
Contributor Author

lowki commented May 13, 2013

C'est quoi ce chemin? /sources/master/

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

C'est mon chemin local vers mes sources : E:\slash-cms\sources\master\slash-cms
J'ai crée un alias Apache :

Alias /slash-cms/ "e:/slash-cms/sources/master/slash-cms/"
<Directory "e:/slash-cms/sources/master/slash-cms/">
Options Indexes FollowSymLinks MultiViews
AllowOverride all
Order allow,deny
Allow from all

@lowki
Copy link
Contributor Author

lowki commented May 13, 2013

Ok je vais essayer avec ça

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

Sinon tant pis, on utilise le RewriteBase.

@wakdev
Copy link
Owner

wakdev commented May 13, 2013

En même temps, c'est un peu son rôle...

@lpotherat
Copy link
Contributor

Je m'incruste,
le RewriteBase peut être généré à l'installation ? ;)
.htaccess n'est qu'un fichier après tout.

@wakdev
Copy link
Owner

wakdev commented May 14, 2013

C'est jouable je pense. ^^

@wakdev
Copy link
Owner

wakdev commented May 14, 2013

Après si il y a moyen de l'éviter, c'est mieux

@lowki
Copy link
Contributor Author

lowki commented May 14, 2013

Oui bien sur c'est possible, mais comme le dit wak, je préfère éviter.
Pour une fois qu'on a le temps, faisons les choses bien :)

@lowki
Copy link
Contributor Author

lowki commented May 17, 2013

Bon finalement j'ai cédé au RewriteBase, je l'ai inclus au script d'installation.
Le pull request arrive.

@wakdev
Copy link
Owner

wakdev commented May 17, 2013

Oui, enfin bon c'est un cas bien particulier, ce n'est pas bien grave.
Merci pour le pull request, j'ai test et merge :)

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.

None yet

3 participants