Misleading popup and inconsistent behaviour with invalid categories

[bfabio]

Hello @AlexPalaz, @valeriocomo this does popup a dialog that says "unknown fields removed: categories", which is misleading. categories is not an unknown field:

Interestingly, if you try and load a file with an invalid category AND a valid one, such as:

categories:
  - no_such_category
  - accounting

it gets autofixed and no dialog is shown.

I think all the "autofixes" are better served shown via the warning pane, so they are uniform and all in the same spot:

Originally posted by @bfabio in #527 (comment)

[valeriocomo]

hi @bfabio .

Popup message

So, if I get it right, you suggest to adopt a less misleading message? What kind of values did you got for categories?

Warning pane

In the warning pane, it shows only warning that come from the validator (go parser). We can think to move all the "autofixes" in the parser and return a message back to the user as warning. What do you think?

[bfabio]

Hi @valeriocomo,

In the warning pane, it shows only warning that come from the validator (go parser). We can think to move all the "autofixes" in the parser and return a message back to the user as warning. What do you think?

Yes, I was suggesting that, but I retried and it looks like you already implemented it, good job! That's way more user friendly.

[valeriocomo]

@bfabio could I close this issue?

[bfabio]

@valeriocomo yes, please