Skip to content

Conversation

@christianclavet
Copy link
Contributor

@christianclavet christianclavet commented Sep 9, 2025

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

…tch output.

Also. Disable the LOAD option when the option is selected.
@@ -1,5 +1,110 @@
<?xml version="1.0" encoding="utf-8"?>
Copy link
Owner

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;
Copy link
Owner

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)

Copy link
Contributor Author

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>
Copy link
Owner

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,
Copy link
Owner

Choose a reason for hiding this comment

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

LoadFromNaps2

Copy link
Contributor Author

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;
Copy link
Owner

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;

Copy link
Contributor Author

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">
Copy link
Owner

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")

Copy link
Contributor Author

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()
Copy link
Owner

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

Copy link
Contributor Author

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,
Copy link
Owner

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)

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