iwyg / filemanager

Filemanager for Symphony CMS
13 stars 11 forks source link

Search + Filebrowser #5

Closed quma closed 12 years ago

quma commented 12 years ago

Hallo, sorry hatte es vorhin bei der falschen Erweiterung gepostet

habe zwei Verbesserungsvorschläge:

1) Suche optional ausschaltbar (Suchfeld nicht anzeigen) 2) die Idee als Standalone Filebrowser ist klasse aber die Umsetzung bzw. die Usability ist nicht optimal.

Danke

iwyg commented 12 years ago

kein Problem.

Punkt 2 hatte ich vor einiger zeit schon gefixt, ist aber noch nicht committet. Über Punkt 1 muss ich noch mal nachdenken. Ich weiß, dass SSM diese Option neuerdings anbietet, daher war die Überlegung auch da (zumindest peripher ).

Aber danke für die Anregung,

iwyg commented 12 years ago

Hallo quma. Mit 1.0.5 sollten die von Dir angesprochenen Dinge implementiert sein. Die abgekoppelte Suchfunktion fand ich dann letztendlich eine gute Entscheidung, danke für das. Würde mich über Feedback freuen.

quma commented 12 years ago

Hallo,

danke für die schnelle Umsetzung! Konnte es allerdings noch nicht testen ... nach dem update erscheint bei mir nur noch die Ladeanzeige und sonst nichts. Sobald ich zeit habe schaue ich mal an was das liegt.

iwyg commented 12 years ago

War ein korrupter commit dabei (wegen Upgrade der lokalen requirejs Version), das sollte aber gefixt sein. Das Phänomen kenne ich allerdings. Falls das den letzten Commit betrifft, versuche bitte mal den Browsercache zu löschen, danach sollte es funktionieren

quma commented 12 years ago

Den Browsercahe zu löschen bringt schon was aber jedes mal wenn man was an den Feldeinstellungen ändert tritt das Problem gleich wieder auf.

Ansonsten ist die Anwendung deutlich besser geworden. Man könnte es eventuell noch etwas verbessern in den man das ein und ausblenden der Dateiinformationen über ein klick auf die Datei steuert. Ist allerdings die Frage wie man dann die Selektion der Datei löst.

iwyg commented 12 years ago

kannst bitte mal die error Logs posten?

quma commented 12 years ago

05 June 2012 15:41 > Error: Javascript - a.deferred is undefined in file .../extensions/filemanager/assets/js/main.js on line 1

iwyg commented 12 years ago

danke

iwyg commented 12 years ago

hm, ich kann den Fehler leider nicht reproduzieren. main.js ist ja in dem fall minified. Du könntest in fielmanager/dev/ mal dev.sh anwerfen. Falls der Fehler noch auftritt sollte die Error Log auch aufschlussreicher sein

quma commented 12 years ago

ich kann die dev.sh nicht anwerfen weil ich kein zugriff drauf habe. Deswegen habe ich den Inhalt von dev in assets kopiert, bringt leider auch keine Änderung.

Zum testen hatte ich zwei Sectionen, beides mit einem Textinput-Feld (Title) und einem Filemanager 1x mit Auswahl und einmal ohne Auswahl. Beide Sektionen waren mit der Erweiterung Limit sections entries auf je ein Eintrag limitiert.

Habe die Erweiterung mal gelöscht, jetzt geht zumindest eine der Sektionen, bei der anderen wird immer die Ladefunktion angezeigt.

Mehr habe ich noch nicht raus gefunden

Hier die Log .. hat sich kaum was geändert: 05 June 2012 23:48 > Error: Javascript - TypeError: a.deferred is undefined in file .../extensions/filemanager/assets/js/main.js on line 1 05 June 2012 23:50 > Error: Javascript - TypeError: manager.deferred is undefined in file .../extensions/filemanager/assets/js/main.js on line 93

iwyg commented 12 years ago

Dachte ich mit dann schon fast, dass es der deferred ist. Tritt das in allen Browsern auf?

quma commented 12 years ago

Du hast recht ist nur im Firefox 13 Beta und 14 Beta so. In Safari und Google Chrome geht es (Mac OsX)

In der Finalen Firefox 13 habe ich es nicht getestet.

iwyg commented 12 years ago

ok, ich habe trotzdem nen Bug gefunden der damit zusammen hängen könnte, betraf in dem Fall allerdings nur Opera. Kannst Du versuchen ob main.js von hier Dein Problem behebt?

quma commented 12 years ago

Habe es gerade getestet, leider hilft es nicht.

Zeitstempel: 06.06.12 01:55:31 Fehler: SyntaxError: syntax error Quelldatei: http:.../extensions/filemanager/assets/js/main.js Zeile: 4 Quelltext: <!DOCTYPE html>

In der Log Datei selbst sehe ich keine neuen Einträge

iwyg commented 12 years ago

hm, <!DOCTYPE html> sollte da nicht auftauchen

iwyg commented 12 years ago

kann ich das schließen, oder tritt es nach wie vor bei dir auf?

quma commented 12 years ago

ja bitte schließen !

iwyg commented 12 years ago

alles klar, weißt Du woran es lag?