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

Optimazion for Contao 3.1 #4

Closed
wants to merge 4 commits into from
Closed

Optimazion for Contao 3.1 #4

wants to merge 4 commits into from

Conversation

iCodr8
Copy link
Contributor

@iCodr8 iCodr8 commented Aug 13, 2013

Mit den Änderungen des Pull Requests funktioniert das Widget auch unter Contao 3.1.

@aschempp
Copy link
Member

Danke. Die htaccess-Datei habe ich übernommen, aber das JS-Problem musste ich anders beheben (siehe 5b139d2). Warum musstest du das ctrl_ vor den Namen stellen? Kannst du mal sehen ob mein aktueller Develop funktioniert?

@iCodr8
Copy link
Contributor Author

iCodr8 commented Aug 13, 2013

Der aktuelle Develop funktioniert bei mir nur, wenn ich ctrl_ von den Name setze. Im HTML Code sieht es bei mir so aus:
screen shot 2013-08-13 at 13 22 34
Im JavaScript Code erhalte ich jedoch durch die Variable name nur mid und nicht ctrl_mid.

@iCodr8
Copy link
Contributor Author

iCodr8 commented Aug 13, 2013

Vielleicht ist es auch sinnvoller das ctrl_ nicht in der JavaScript Datei zu setzen, sondern lieber in der TableLookupWizard.php.
screen shot 2013-08-13 at 14 09 45

@iCodr8
Copy link
Contributor Author

iCodr8 commented Aug 13, 2013

Ah nein, das geht auch nicht, denn dann wird das ctrl_ auch an den hrefdran gehängt.
Das ctrl_muss also doch nur in der JS Datei bei document.id(... dran gehängt werden.

@iCodr8
Copy link
Contributor Author

iCodr8 commented Aug 15, 2013

Gibt's schon was neues?

@aschempp
Copy link
Member

@Toflar kannst du dir das ansehen? Ev. relevant für Isotope 2.0

@Toflar
Copy link
Member

Toflar commented Sep 23, 2013

Hab's mir angesehen. Hab die aktuelle Version (1.3.5) getestet und sie funktioniert einwandfrei. Ich kann zudem im Code keine Stelle finden, an der das ctrl_ falsch gesetzt wäre. @iCodr8 was funktioniert denn nicht? Wie kann ich es reproduzieren? Wo ist das Problem deiner Meinung nach in der 1.3.5 genau (welche Zeile)?

@Toflar Toflar closed this Oct 18, 2013
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