Skip to content

enabled for ID image file upload - #116

Open
nielsson4711 wants to merge 2 commits into
mainfrom
id_by_file
Open

enabled for ID image file upload#116
nielsson4711 wants to merge 2 commits into
mainfrom
id_by_file

Conversation

@nielsson4711

Copy link
Copy Markdown
Collaborator

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.

@patrickuhlmann patrickuhlmann left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

http://localhost:5173/#%7B%22langUi%22:%22de%22,%22langCor%22:%22de%22,%22v%22:1,%22entry%22:%22followup%22,%22desire%22:%22data_info_request%22,%22name%22:%22asdf%22,%22address%22:%22sdf%22,%22org%22:%22Agrisano%22,%22orgAddressEntry%22:%22sdfsdf%22,%22step%22:%22print%22,%22date%22:%2222.4.2026%22%7D

(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.

@nielsson4711

Copy link
Copy Markdown
Collaborator Author

:-) ganz sicher müssen wir da nochmal drüberschauen.

@patrickuhlmann

patrickuhlmann commented Apr 23, 2026

Copy link
Copy Markdown
Collaborator

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

@patrickuhlmann

Copy link
Copy Markdown
Collaborator

Not planned for the current August/September release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants