Skip to content

Conversation

obreidenich
Copy link

Added default appearance definitions in examples/forms to match expected behaviour, as without them, text to be displayed disappears after input until the document is saved and reloaded.

@laurmaedje laurmaedje changed the title Added default appearance definitions in examples/forms to match expected behaviour. Added default appearance definitions in examples/forms.rs to match expected behaviour. Jul 1, 2025
@laurmaedje
Copy link
Member

I could detect no difference in Acrobat and Preview. Could you explain in more detail what this is supposed to fix or share a screencast?

@laurmaedje
Copy link
Member

I'm not that familiar with PDF forms. The example was contributed by @tingerrr. Maybe he knows more?

@tingerrr
Copy link
Contributor

tingerrr commented Aug 1, 2025

I'm not sure, I don't remember that being a problem for input fields in the viewers I tried, but it has been a while, so this could have changed.

@tingerrr
Copy link
Contributor

tingerrr commented Aug 1, 2025

Ah, before this patch both evince (gnome) and okular (KDE) haven't properly displayed the content in the form fields, this patch fixes it. Though, this seems to only fix it for the text field at the top for me, the rest is still hardly visible (in Okular this is fixed by clicking "show forms", in evince they're just invisible).

@laurmaedje
Copy link
Member

Though, this seems to only fix it for the text field at the top for me, the rest is still hardly visible (in Okular this is fixed by clicking "show forms", in evince they're just invisible).

@obreidenich Any thoughts on this?

@obreidenich
Copy link
Author

Sorry for the long absence.

Yes, this only fixes the appearance stream on Okular and, frankly, whatever PDF reader i was using on Android at the time.

I just assumed this was a forgotten line/feature when diving into krilla/forms.

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.

3 participants