Added an option to ignore General Macros with import/export

[pgarneau]

Hello!

I was finding it frustrating that I was losing general macros on characters that didn't use Myslot when using the ClearMacro functionality.

Added an option to ignore general macros and only import/export/clear character specific macros

[pgarneau]

@tg123

[tg123]

the ui is confusing,

do you mind change it to

Ignore Import Export [Macro drop down] 
                                        - All Macro
                                        - General
                                        - Character specific

also i am thinking to support

Ignore Import Export [Action] 
                                        - All
                                        - 1
                                        - 2
                                        - 3
                                        - ...
                                        - 12

[pgarneau]

You want one single drop down menu for all ignore import/export?

What about the clear section?

You will have one side of the UI that will be small and other side will be still large

[tg123]

agreed, any idea to make the ui better to understand?

[tg123]

first POC

still too complex

[pgarneau]

Could Do something like this:

Myslot is already a complex addon, I think this would be fine.

   Ignore during Import/Export: DROPDOWN                       Clear during Import: DROPDOWN
                                        -> Action Bars                                                             -> Action Bars
                                        -> Key Bindings                                                           -> Key Bindings
                                        -> General Macros                                                       -> General Macros
                                        -> Character Specific Macros                                 -> Character Specific Macros                                                                             

[tg123]

will draft ui as you suggested and lets work together to close this

[pgarneau]

will draft ui as you suggested and lets work together to close this

Working on it at the moment. I found a bug in the "ignore general macro" where it wouldn't reimport the general macro on the action bars. Fixed it, but it's not super elegant. I'm trusting the index stored in the action bar data. What if that general macro got deleted and a new one is there? The "placed" macro will be wrong. Might add some logic that checks if the macro name exists or something.

Here's a quick draft

[pgarneau]

@tg123 Thoughts?

[pgarneau]

I think we should make the text window smaller, but keep the old checkboxes.

More clear what options are enabled/disabled at the time of import.

Nobody reads the text anyways

[pgarneau]

you can add me on discord if you want to chat more easily:

username: doll