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

Mein Versuch etwas beizutragen #12

Closed
obenland opened this Issue Feb 1, 2012 · 2 comments

Comments

Projects
None yet
3 participants

obenland commented Feb 1, 2012

Hallo zusammen,

ich probiere gerade euer Plugin aus und hätte da ein paar Vorschläge/Feature Requests/Bugs bzgl. des Widgets zu reporten :) Würde mich freuen, wenn ich auf diesem Wege zum Plugin beitragen darf:

  • Variablen als Textdomain sind wohl nicht soo toll
  • Überschriften sind keine Attribute esc_attr() macht hier also glaub ich nicht so viel Sinn wie z.B. strip_tags() (Z. 36,66)
  • Aktuell wird beim Update 'widget_link_type' nur übernommen, wenn eine überschrift eingegeben wurd (Z. 37)
  • Die selected() Funktion könnte u.U. die options im select etwas aufräumen (Z. 47ff)
  • Anonyme Funktionen kann man mit remove_action nicht entfernen - ist das so gewollt? (Z. 136)

Zur Anzeigelogig des Widgets:
Wenn ich auf einer Einzelseite bin, macht es dann nicht Sinn die Flaggen sich auch nur auf diesen Inhalt beziehen zu lassen? Aktuell ist es ja so, dass die Flagge der Sprache die ich mir gerade ansehe auf die Startseite verlinkt. Das ist - zumindest bei mir - absolut unerwartetes Verhalten.
Erwartet hätte ich, dass die Flagge entweder garnicht angezeigt wird (denn ich bin ja eh schon da) oder wenigstens auf den gleichen Artikel verlinkt, wenn sie schon verlinkt.
Auch auf Suchergebnisseiten, paginierten Seiten, Kategorieseiten (ihr wisst worauf ich hinaus will :) ) wird nur auf die Startseiten verlinkt statt auf die Dependents in der anderen Sprache.

Abschließend möchte ich mich dafür bedanken, dass ihr dieses Projekt ins Leben gerufen habt und hier soviel Zeit investiert! Ich finde das Plugin, die Idee dahinter und die Umsetzung absolut großartig! Dankeschön!

Owner

nullbytes commented Feb 2, 2012

Deine Punkte bis auf die Anzeigelogik wurden mit Version 0.7.5a behoben. Die Anzeigelogik werden wir in der nächsten Version anpassen.

@nullbytes nullbytes was assigned Feb 2, 2012

Owner

bueltge commented Feb 2, 2012

ich übernehme die Widgetlogik-Idee in eine neuen Issue, Übersicht.
Die Var für Sprachdateien bleibt, dass passt bei allen Tests super und ist sauberer zu pflegen; der Artikel ist bekannt :(

Merci für die Meldungen! Umsetzung in 0.7.5a

@bueltge bueltge closed this Feb 2, 2012

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