-
-
Notifications
You must be signed in to change notification settings - Fork 394
Batch Scanning, added an option to output the images content of NAPS2 from this panel. #696
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…tch output. Also. Disable the LOAD option when the option is selected.
| @@ -1,5 +1,110 @@ | |||
| <?xml version="1.0" encoding="utf-8"?> | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't include the changes to localized files, these will be updated as part of the release process
| private readonly Button _start = new() { Text = UiStrings.Start }; | ||
| private readonly Button _cancel = new() { Text = UiStrings.Cancel }; | ||
| private readonly DropDownWidget<ScanProfile> _profile = new(); | ||
| private readonly RadioButton _UseExistingImages; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should start with a lowercase letter to match, but also let's rename this to _loadFromNaps2 (and, optionally, you can also rename _load to _loadIntoNaps2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed.
| <EmbeddedResource Remove="ClientCreds_.resx" /> | ||
| <Compile Remove="ClientCreds_.Designer.cs" /> | ||
| </ItemGroup> | ||
| <ItemGroup> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes also shouldn't be included
|
|
||
| public enum BatchScanType | ||
| { | ||
| UseExistingImages, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LoadFromNaps2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
| using Eto.Forms; | ||
| using Eto.Forms; | ||
| using NAPS2.EtoForms; | ||
| using NAPS2.EtoForms.Desktop; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only change to usings here should be adding using NAPS2.Images;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I touch theses, lots of errors pop out, I cannot remove them. using NAPS2.Images is there but it's greyed as if it was not used.
| <data name="StopScannerSharing" xml:space="preserve"> | ||
| <value>Stop Scanner Sharing</value> | ||
| </data> | ||
| <data name="UseExistingImages" xml:space="preserve"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's call this LoadFromNaps2 with a string "Load images from NAPS2" (to be consistent with the output string of "Load images into NAPS2")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
| ShowUpdatePrompt(); | ||
| } | ||
|
|
||
| public UiImageList GetImages() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be necessary any more
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, removed. There doesnt seem to have any reference to it.
| L.GroupBox( | ||
| UiStrings.ScanConfig, | ||
| L.Column( | ||
| _UseExistingImages, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move this to the bottom of the column (below the ).Padding(left: 20).Visible(_delayVis) line)
Hi, This is a new option to the Batch Scanning panel.
It provide a way to use the already scanned images and use the output functions. Right now, it can become useful, if you scanned images with PATCH "T" symbol. So it enable the use to scan multiple documents, do some work over them and export them as multiple documents with the Patch "T" images.
When the option is checked it will disable the LOAD option when the option is selected since It would loop the data between what is in NAPS and what is put in NAPS2. The LOAD option will become enabled again, if the user selection another option.
Later On, I plan to add more features to this panel. Output a text containing a filelist, that could be used a signal to allow other software to get the image and use NAPS2 as the document scanner part of the document solution.
NOTE: The changes affect only the Windows build for the moment, the GUI work for MAC and Linux need to be done.
Thanks
Christian Clavet
Montreal