Uniformation of Edit Properties and detail card

[vlt1]

Description

Replaces MetadataModal with existing general SaveModal in MapGrid. Adds details sheet to that SaveModal. Also should solve this behaviour.

Please check if the PR fulfills these requirements

• The commit message follows our guidelines: https://github.com/geosolutions-it/MapStore2/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x", remove the others)

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe: enhancement

Issue

#5772

What is the current behavior?

#5772

What is the new behavior?
MetadataModal in MapGrid is removed and replaced with SaveModal. SaveModal is extended with details sheet.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

• Yes, and I documented them in migration notes
• No

[coveralls]

Coverage decreased (-0.3%) to 83.259% when pulling d9dc1de on vlt1:enhancement-5772 into 3b0bf21 on geosolutions-it:master.

[offtherailz]

• Great! We cleaned up a lot. I'd like to clean up even more. I think we can reuse almost everything from the ResourceGrid. Please look at my inline comments for details

• I see there is an undo button when you create a new detail card that should not appear. Probably is due to the NODATA handling I mentioned
  

• If I remove from the FeatureGrid the detail card, I still see the detail button. Clicking on it causes shows a "NODATA"
  

• Please add unit tests for new things you wrote.

[vlt1]

@offtherailz when should the undo button appear

[offtherailz]

@vlt1 checking the current behaviour, only when the user deleted the detail card, to restore it. In fact I see the text is "undo remove"

[vlt1]

@offtherailz if rework is good I'll add tests

[offtherailz]

@vlt1 looks really great! 👍 🚀
Please move testing the new/uncovered parts (they should be not that much).
As usual tests that make sense the functionalities and corner cases of units. ;) We already increased a little the coverage just removing unuseful code.

https://coveralls.io/builds/33329679