Stop hiding the grids when POI is removed

[MizukiTemma]

Short description

This PR fixes the prblem that the middle and right grid are closing when a POI is removed while editing an existing contact.

Proposed changes

• Remove code lines that close the grids whenever location is unseleted
• Change the trash icon to pencil icon
• Add a message indicating location is mandatory.

Side effects

• I hope none 🙈
• Please check for side effects in event form too

Resolved issues

Fixes: #3151

---

Pull Request Review Guidelines

[JoeyStk]

Thank you very much for the PR! I think it looks good so far 🚀

During my review I checked the following:
✔️ The PR follows the design and uses the correct wording of the design
✔️ Code is understandable to me (besides one nit and one question, see below :) )
✔️ Checking in the events form for side effects. By positive testing, I couldn't find a side effect. The next reviewer could focus on some negative testing :)

[MizukiTemma]

@JoeyStk Thank you for review 😸 Your suggetions are applied 👍

[david-venhoff]

Thanks, looks good to me.

One thing I noticed is that the grids still get hidden if the user clicks submit when the location is empty. Even though, when the user selects a location again, the previous values for the other grid items are still there.
Do you think it would make sense to also fix this?

[MizukiTemma]

One thing I noticed is that the grids still get hidden if the user clicks submit when the location is empty. Even though, when the user selects a location again, the previous values for the other grid items are still there. Do you think it would make sense to also fix this?

@david-venhoff Yes 😸 patched to keep them open if any of the five fields is filled with something.