Skip to content

Conversation

@uriel1998
Copy link

Added backup way of finding thumbnails that works on non-gnome system
Added clobbering check for automatic renaming based on metadata
Added optional large thumbnails for PDF files if poppler-utils and imagemagick are present
Added check for OCR layer (using poppler-utils) and checkbox to perform OCR on exit (using ocrmypdf)
Removed "open folder" button (because I was getting lost in what form field went where trying to add one)
Added new screenshot with large preview and OCR button

**Screenshot**

![screenshot of PDFMtEd Editor](https://github.com/Glutanimate/PDFMtEd/blob/master/screenshots/pdfmted-editor.png)
![screenshot of PDFMtEd Editor](https://github.com/uriel1998/PDFMtEd/blob/7f96c7170e6fc66992635cb1bd94ba824ffa790d/screenshots/pdfmted-editor.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is incorrect. It should lead to the screenshot in the original repo.


- easily process multiple files and folders right from your file manager
- preview your documents in your default PDF viewer and file manager before editing them
- preview your documents in your default PDF viewer before editing them
Copy link
Contributor

Choose a reason for hiding this comment

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

Obviously, I'm not the maintainer but I don't think removing the browse button is a good idea.


sudo apt install libimage-exiftool-perl yad qpdf

If you do not have Gnome installed, you may need `gir1.2-gnomedesktop-3.0` and `gir1.2-gnomedesktop-4.0` for thumbnail generation.
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps find a way to implement the feature without it depending on Gnome?

@ifohancroft
Copy link
Contributor

P.S. Implementing multiple unreleated features in a single PR is a bad idea and a frowned upon practice. Please submit separate PRs for the separate features.

Disclaimer: I'm not the maintainer.

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