Skip to content

Add Partially scanned PDFs preferences section and OCR shortcut - #642

Merged
Siedlerchr merged 11 commits into
JabRef:mainfrom
ZiadAbdElFatah:more-preferences
Jul 29, 2026
Merged

Add Partially scanned PDFs preferences section and OCR shortcut#642
Siedlerchr merged 11 commits into
JabRef:mainfrom
ZiadAbdElFatah:more-preferences

Conversation

@ZiadAbdElFatah

Copy link
Copy Markdown
Contributor

Comment thread en/advanced/OCR.md Outdated
![ocr-preferences.png](../.gitbook/assets/ocr-preferences.png)
![OCR preferences](../.gitbook/assets/ocr-preferences.png)

### Partially scanned PDFs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Alternatives:

  1. handling of pre-existing text
  2. handling of original embedded text
  3. handling of text input
  4. text creation handling
  5. handling of text creation
  6. text flow handling
  7. handling of source text

I am not quite sure what is best, but tend to favour 1 or 2. What do you think?
"Partially scanned PDFs" doesn't seem to be entirely correct. First of all we are not scanning. Secondly, overwriting is not handling it partially.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I would go with option 1.
Unfortunately I don't have access to my laptop right now.

Comment thread en/advanced/OCR.md

![ocr-preferences.png](../.gitbook/assets/ocr-preferences.png)
![OCR preferences](../.gitbook/assets/ocr-preferences.png)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have seen other apps structure OCR preferences via "Input" vs "Output".

Comment thread en/advanced/OCR.md Outdated
Comment thread en/advanced/OCR.md Outdated
Comment thread en/advanced/OCR.md Outdated
Comment thread en/advanced/OCR.md Outdated
ZiadAbdElFatah and others added 5 commits July 17, 2026 02:16
Co-authored-by: ThiloteE <73715071+ThiloteE@users.noreply.github.com>
Co-authored-by: ThiloteE <73715071+ThiloteE@users.noreply.github.com>
Co-authored-by: ThiloteE <73715071+ThiloteE@users.noreply.github.com>
Co-authored-by: ThiloteE <73715071+ThiloteE@users.noreply.github.com>
@ZiadAbdElFatah
ZiadAbdElFatah requested a review from ThiloteE July 29, 2026 11:10
Comment thread en/advanced/OCR.md

@InAnYan InAnYan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good! But I don't know whether the images should be both in .gitbook/assets or advanced

@InAnYan

InAnYan commented Jul 29, 2026

Copy link
Copy Markdown
Member

I think it (images) should all go to gitbook assets

@InAnYan

InAnYan commented Jul 29, 2026

Copy link
Copy Markdown
Member

Uploading {413DBA77-9291-40CA-9186-9418EB336FBD}.png…

Many deleted, but they are present in .gitbook/assets?

@Siedlerchr

Copy link
Copy Markdown
Member

Should be fine now. We will see after merge. We also have now gitbook skills in place in the repo e.e.g https://github.com/GitbookIO/gitbook-skills/blob/main/skills/write-docs/SKILL.md

@Siedlerchr
Siedlerchr merged commit 7a05444 into JabRef:main Jul 29, 2026
6 checks passed
@ZiadAbdElFatah
ZiadAbdElFatah deleted the more-preferences branch July 29, 2026 21:34
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.

5 participants