Feature/delete multiple feature

[kaustuvpokharel]

Case:

Multiselect feature exists, but there is no way to delete all selected multiple feature at once, so pr includes the deleteMultipleSelect feature

Fix:

• Included feature deletion inside MultiEditManager class
  //! Deletes selected or toggled features upon called Q_INVOKABLE void deleteSelectedFeature();
• Changed UI with delete button with respected UI
• Added the drawer to verify the deletion before actually deleting it

Tests:

• Added test cases only for checking deletion feature inside the new file testmultieditmanager
  Note: this file is incomplete in a sense that it is missing other feature test cases which already existed inside the MultiEditManager class, as I was only requested to add tests for my new feature addition. But it needs more test cases to test other prior existing functions/methods.

[github-actions]

Pull Request Test Coverage Report for Build 18636599772

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 1014 unchanged lines in 7 files lost coverage.
• Overall coverage increased (+0.1%) to 19.756%

---

Files with Coverage Reduction	New Missed Lines	%
mm/app/attributes/attributecontroller.cpp	1	76.83%
mm/app/layerfeaturesmodel.cpp	1	79.5%
mm/core/merginapi.cpp	1	75.22%
mm/app/multieditmanager.h	2	0.0%
build-mm-db/app/Input_autogen/EWIEGA46WW/moc_multieditmanager.cpp	25	15.31%
mm/app/multieditmanager.cpp	42	50.4%
build-mm-db/app/.rcc/qmlcache/Input_qml/main_qml.cpp	942	0.0%

Totals
Change from base Build 18284755654:	0.1%
Covered Lines:	14047
Relevant Lines:	71104

---

💛 - Coveralls

[tomasMizera]

Tests + updates as discussed :)

[kaustuvpokharel]

[kaustuvpokharel]

Screen.Recording.2025-10-01.at.8.09.53.PM.mov

[Withalion]

I still believe we can do better than using (s) in the text, maybe propagate through the signal if it's single or multi edit and then use ternary operator inline.

[tomasMizera]

I still believe we can do better than using (s) in the text, maybe propagate through the signal if it's single or multi edit and then use ternary operator inline.

https://doc.qt.io/qt-6/i18n-source-translation.html#handle-plural-forms

[tomasMizera]

Hi @kaustuvpokharel - we can close the multiselect drawer after features are deleted.

[kaustuvpokharel]

Noted

Dialog is not closed after deletion is done

[Withalion]

I would also add test for editing some points not just deleting

[tomasMizera]

Moving to 2025.8.0

[Withalion]

Just few more things

[Withalion]

Seems alright now, let's see what @tomasMizera thinks.

[tomasMizera]

Just a couple of minor comments :)

[tomasMizera]

Let's hope screens are wide enough to incorporate both buttons without the need to scroll (in all languages 🙃)