Delete jewels, skills, and equipment by default

[raylu]

When importing a character, default the 3 delete checkboxes to checked.

Description of the problem being solved:

Just an inconvenience since you generally are not importing characters onto builds with existing content.

Steps taken to verify a working solution:

• Imported a character twice
• Checked the skills tab "Sort gems by DPS" was still checked and adding a new support gem offered reasonable options

Before screenshot:

After screenshot:

[pHiney]

Can the 'Cancel' button be 'Close' instead (just to hijack the PR) ?

[raylu]

switched "Cancel" to "Close" in #3939 instead

[mightyscrub]

"Just an inconvenience since you generally are not importing characters onto builds with existing content." Thats not true in all cases. I'm doin that all the time (and was really confused until i found this topic - i managed to miss that in the change log).

Maybe you could add an option to the configuration menu so users can pick "their favorite default".

[raylu]

could you elaborate on your use case?

[mightyscrub]

If you are sure it's only "my" use case you shouldn't change anything. I can deal with it. But in my opinion it's not ideal to change defaults, not even to set defaults, without the option that users can override.

[raylu]

it would be informative for us, regardless of whether it changes, if you elaborated on your use case. how did you get into a situation where you have items but want to import a character?

[mightyscrub]

On league start i merge my new character into the old one (if it's "the same" build)

I also import my own characters periodically several times during "progress", keeping gear and jewels. Mostly to keep some kind of history for future uses (above-mentioned). Sometimes only to check which combination will deal more damage. (I know, that it's possible to copy stats form in-game-item and past them into pob. But "my way" feels more convenient.)