Closed OliEfr closed 2 years ago
Ich konnte soweit alle Bugs bestätigen.
Außer: zumindest optisch werden die E-Mail Einstellungen übernommen (d.h. die Haken sind dann auch auf der Settings-Seite gesetzt). Ob die Funktionalität geht habe ich nicht überprüft.
Ich habe gerade noch gemerkt, dass die Einstellungen für die E-Mails in Realität irgendwie garnicht übernommen werden. Ich habe alles deaktiviert, sehe aber trotzdem die Anzahl meiner ungelesenen Mails im Icon.
- Bei der Anmeldekarte könnten wir vllt mindestens ein Feld befüllen. Es ist nicht so ganz offensichtlich, dass da was gespeichert ist
Der grüne Text "aktiv" bzw rot "nicht aktiviert" findest du reicht nicht?
- die Einstellungen für die E-Mail werden irgendwie nicht vom onboarding übernommen. Was weird ist, weil er fragt nach Berechtigungen und das kommt aus dem Backend? Das muss ich auf jeden Fall nochmal checken
@C0ntroller ist das geklärt, oder muss ich da meinerseits noch was machen?
- [x] selbe Karte: Button fürs Löschen entweder auffälliger machen (weil wir ohne nochmal zu fragen löschen) oder nen prompt
Was sagst du zur schönen roten Farbe, die der Button jetzt hat?
- Bei der Anmeldekarte könnten wir vllt mindestens ein Feld befüllen. Es ist nicht so ganz offensichtlich, dass da was gespeichert ist
Der grüne Text "aktiv" bzw rot "nicht aktiviert" findest du reicht nicht?
Mhh vllt könnten wir das auch in die Placehholder der Inputfelder schreiben? Dann kann man das quasi nicht übersehen :D
- die Einstellungen für die E-Mail werden irgendwie nicht vom onboarding übernommen. Was weird ist, weil er fragt nach Berechtigungen und das kommt aus dem Backend? Das muss ich auf jeden Fall nochmal checken
@C0ntroller ist das geklärt, oder muss ich da meinerseits noch was machen?
Bin leider noch nicht zu gekommen :/ Also besteht noch weiterhin. Aber möglicherweise hab ich irgendwas im Backend kaputt gemacht :/
[x] die Listen für Suchmaschinenabkürzungen sind unterschiedlich. Bei der Liste im Onboarding fehlen slub und videocampus und OPAL ist noch groß.
[x] Vielleicht könnten wir auch die Liste aus src/contentScripts/forward/searchEngines/common.ts irgendwie auslagern und dynamisch importieren?
[x] TUfast ist teils TUFast etc. Das alles gleich machen
[ ] Bei der Anmeldekarte könnten wir vllt mindestens ein Feld befüllen. Es ist nicht so ganz offensichtlich, dass da was gespeichert ist
[x] selbe Karte: Button fürs Löschen entweder auffälliger machen (weil wir ohne nochmal zu fragen löschen) oder nen prompt
[x] immer noch selbe Karte: unten ein kleines bisschen padding. Wenn es zu klein wird, liegt die vertikale scrollbar schon fast auf dem input
[x] die Einstellungen für die E-Mail werden irgendwie nicht vom onboarding übernommen. Was weird ist, weil er fragt nach Berechtigungen und das kommt aus dem Backend? Das muss ich auf jeden Fall nochmal checken
[x] Ganz wichtig: das Theme wird nicht gespeichert. Hab ich da was kaputt gemacht, als ich useChrome überall benutzt habe?