enabled for ID image file upload - #116
Conversation
patrickuhlmann
left a comment
There was a problem hiding this comment.
Ich habe den Code noch nicht im Detail angeschaut und es einfach mal auf der Benutzeroberfläche durchgeklickt.
Wenn ich mit "Brief Auskunftsbegehren" starte, die Felder ausfülle und den Brief generiere ist erstmal alles gut. Wenn ich dann aber auf "Jetzt drucken" klicke zeigt es mir zwei Briefe an "Datenauskunftsbegehren" und auch "Datenauskunftsbegehren / Ausbleibende Auskunft" und jetzt zeigt es auch beide auf der Benutzeroberfläche an. Bin nicht sicher, ob das allenfalls bereits vorher ein Bug war und nichts mit dem PR zu tun hat.
Wenn ich die URL
(die hat es mir beim durchklicken gemacht) aufrufe und dann "Ausweis doppelseitig" und einen Ausweis ist alles ok. Wenn ich dann aber auf löschen klicke und nochmal auf "Ausweis doppelseitig" dann wird es bisschen komisch. Der Ausweis wird zwar angezeigt aber die Auswahl bleibt bestehen.
Wenn ich wieder die URL aufrufe. Dann "Ausweis doppelseitig", ein Bild auswähle und wieder lösche und danach "Ausweis Vorderseite" auswähle, dann kann ich das Bild auswählen aber es zeigt nichts an. Ich kann dann auch noch "Ausweis Rückseite" auswählen und dann zeigt es wieder nichts an.
Ich denke da ist noch etwas nicht ganz gut mit dem Handling (vermutlich wird beim Löschen nicht alles sauber zurückgesetzt)
Beim Start zeigt es mir zudem noch einige Warnungen an (hatte mal alle bestehenden Warnungen entfernt, denke daher, dass diese neu sind)
8:36:49 PM [vite-plugin-svelte] src/letter/LetterDataInfoReq.svelte:245:12 Screenreaders already announce `<img>` elements as an image
https://svelte.dev/e/a11y_img_redundant_alt
8:36:49 PM [vite-plugin-svelte] src/letter/LetterDataInfoReq.svelte:251:14 Screenreaders already announce `<img>` elements as an image
https://svelte.dev/e/a11y_img_redundant_alt
8:36:49 PM [vite-plugin-svelte] src/letter/LetterDataInfoReq.svelte:257:14 Screenreaders already announce `<img>` elements as an image
https://svelte.dev/e/a11y_img_redundant_alt
8:36:49 PM [vite-plugin-svelte] src/letter/LetterDataInfoReq.svelte:288:2 Unused CSS selector ".import-container"
https://svelte.dev/e/css_unused_selector
8:36:49 PM [vite-plugin-svelte] src/letter/IdFileImport.svelte:110:10 Visible, non-interactive elements with a click event must be accompanied by a keyboard event handler. Consider whether an interactive element such as `<button type="button">` or `<a>` might be more appropriate
https://svelte.dev/e/a11y_click_events_have_key_events
8:36:49 PM [vite-plugin-svelte] src/letter/IdFileImport.svelte:110:10 `<span>` with a click handler must have an ARIA role
https://svelte.dev/e/a11y_no_static_element_interactions
8:36:49 PM [vite-plugin-svelte] src/letter/IdFileImport.svelte:159:0 Unused CSS selector ".import-step .controls p:first-child"
https://svelte.dev/e/css_unused_selector
8:36:49 PM [vite-plugin-svelte] src/letter/IdFileImport.svelte:232:0 Unused CSS selector ".small .small .image-container img"
https://svelte.dev/e/css_unused_selector
Glaube da müssen wir nochmal etwas darübergehen.
|
:-) ganz sicher müssen wir da nochmal drüberschauen. |
|
Ich habe es unterdessen auch auf main getestet und konnte den Bug mit den zwei Briefen nauch dort nachvollziehen. D. h. es hat nichts mit den Änderungen hier zu tun. Ich habe einen PR erstellt für den Bugfix: #120. Wenn dieser gemerged ist können wir diesen Branch rebasen |
|
Not planned for the current August/September release |
the letter created allows for uploading ID images either as combination of front and back, or as separate image each. Selecting one or another upload option, the file browser opens to support ID image selection from the users local file system. The image selected is renedered into the page. It can easily be removed by clicking the "trashbin" icon shown beneath the image canvas. None of the images are mandatory in either combination.