Skip to content
This repository was archived by the owner on Sep 2, 2021. It is now read-only.
This repository was archived by the owner on Sep 2, 2021. It is now read-only.

Mein Versuch etwas beizutragen #12

@obenland

Description

@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!

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions