Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Mein Versuch etwas beizutragen #12

Closed
obenland opened this Issue · 2 comments

3 participants

@obenland

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!

@nullbytes
Owner

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
@bueltge
Owner

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.