glueckpress/stringintelligenz

On-Boarding

glueckpress opened this issue · 14 comments

glueckpress commented

Menschen, die das Plugin aktivieren, sollten wir pro-aktiv abholen. Dafür gibt es 2 relevante Szenarien:

Unterstützte Locale

  • Das Plugin wurde aktiviert, die Locale wird unterstützt, es sind also Änderungen im UI sichtbar.
  • $user sieht anstelle des üblichen Plugin aktiviert eine Erfolgsnachricht mit kurzen Tipps zu sichtbaren Änderungen, sowie Channels für Feedback und Support.
  • Die Notice braucht nicht lokalisierbar zu sein, weil sie eh nur erscheint, wenn das ganze Backend auf Deutsch angezeigt wird.
  • Die Notice kann weggeklickt werden und bleibt dann weg.

Nicht unterstützte Locale (inklusive anderer Locales als Deutsch)

  • Das Plugin wurde aktiviert, die Locale wird nicht unterstützt, es sind keine Änderungen im UI sichtbar.
  • $user sieht zusätzlich zum üblichen Plugin aktiviert eine Warnung mit Tipps zum Wechseln der Locale im Backend (Einstellungen→Allgemein).
  • Die Notice ist lokalisierbar; default ist Englisch (wie WP selbst).
  • Die Notice kann weggeklickt werden und bleibt dann weg.
glueckpress commented

@tfrommen Bist du so nett und machst die Klasse entsprechend fit? :) Ich liefere dann gerne die Inhalte nach.

tfrommen commented

@glueckpress und du bist wieder am Zug. :)

Ich habe die Admin-Notice-Klasse konfigurierbar gemacht, und bereits die entsprechenden Templates erstellt. Du musst nun eigentlich nur noch die Datei templates/admin-notice-plugin-activated.php entsprechend aufbereiten.

  • 👍1
glueckpress commented

@tfrommen Merci! Ich glaube, wir haben ein Kodierungsproblem? So kamen deine Commits bei mir an:

bildschirmfoto 2016-09-25 um 15 06 06

Bei mir waren das Sonderzeichen und Emoji gewesen.

Ist mir noch nie passiert, dir?

tfrommen commented

Hm, nein. Kannst du das wieder geradebiegen?

glueckpress commented

Klaro, schon geschehen, kein Ding. ;)

Viel interessanter ist die Sache mit „Notice kann weggeklickt werden und bleibt dann weg“. Hast du zufällig eine schicke Routine rum liegen, die das native is-dismissible dafür nutzt?
(Nach kurzer Googelage: Sowas vielleicht? https://github.com/collizo4sky/persist-admin-notices-dismissal)

tfrommen commented

Viel interessanter ist die Sache mit „Notice kann weggeklickt werden und bleibt dann weg“. Hast du zufällig eine schicke Routine rum liegen, die das native is-dismissible dafür nutzt?

Nein, aber das ist nicht besonders schwer. Die Sache ist nur, dass wir dadurch definitiv eine JavaScript-Datei und (wenn man es richtig macht) zwei neue PHP-Dateien einführen. Was natürlich kein Problem ist.

@glueckpress Soll ich mich, bei Zeit, darum kümmern?

glueckpress commented

Soll ich mich, bei Zeit, darum kümmern?

@tfrommen Das wäre klasse, danke dir! Wenn ich es mache, zerhaue ich das schöne OOP. ;)

glueckpress commented

Mist, Issue zu taggen vergessen. :/
ac8526a

tfrommen commented

Wassh, ich hab das schon zu 90 % fertig, musste eben aber weg...

  • 🎉1
glueckpress commented

Revert…!!!

glueckpress commented

Reverted… 94f26f7

  • 😄1
tfrommen commented

@glueckpress here you are.

Ich bin mir nicht sicher, ob das Verhalten so ist, wie du es dir vorgestellt hast, daher schau einfach mal rein...
Falls du ein Notice wieder reaktivieren willst, musst du auf der entsprechenden Site die entsprechende Option stringintelligenz_admin_notice_SOME-HASH-HERE löschen.

  • 👍1
glueckpress commented

@tfrommen Getestet, alles tadellos. 👍 Wollen wir das mergen, oder bist du noch an irgendwas zugange?

tfrommen commented

@glueckpress wenn alles funktional deinen Wünschen entpsricht, kann das gemerget werden, klar.

  • ❤️1