Change the default value of atk, def to Empty (-1)

[salix5]

No description provided.

[Copilot]

⚠️ Not ready to approve

The UI currently displays unknown ATK/DEF (-2) as “-2” instead of “?”, and the updated readmes don’t fully document the new blank/0 ATK/DEF search semantics.

Pull request overview

This PR updates DataEditorX to treat ATK/DEF default values as “empty” (-1) in the editor, and adjusts search/save behavior so blank ATK/DEF inputs are interpreted as “no filter” while still supporting unknown (-2 / ?) values.

Changes:

• Introduces Card.Empty and changes Card’s default atk/def initialization to -1.
• Updates the editor UI to display blank ATK/DEF and ID/alias when values are “empty”/zero, and updates parsing so blank ATK/DEF maps to -1.
• Updates database parameter binding and search condition building to account for the new “empty” sentinel behavior.

File summaries

File	Description
DataEditorX/readme.txt	Updates Chinese documentation for ATK/DEF search behavior (currently incomplete for new semantics).
DataEditorX/readme_english.txt	Updates English documentation for card search (currently missing key cases; also contains a typo).
DataEditorX/DataEditForm.Designer.cs	Removes default "0" text values from ATK/DEF/ID/alias textboxes.
DataEditorX/DataEditForm.cs	Implements new empty/unknown parsing and blank rendering for ATK/DEF and IDs; improves binding update for strs.
DataEditorX/Core/Database.cs	Maps -1 to 0 on write; updates ATK/DEF query conditions to treat -1 as “no filter”.
DataEditorX/Core/Card.cs	Adds Card.Empty and changes default atk/def initialization to -1.

Copilot's findings

Files not reviewed (1)

• DataEditorX/DataEditForm.Designer.cs: Generated file

• Files reviewed: 5/6 changed files
• Comments generated: 5

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.