Management of installed maps

[Marc-Spector]

Closes #2026
Closes #1102

[codecov]

Codecov Report

Merging #2079 (a1d867e) into develop (0b21de7) will increase coverage by 0.06%.
The diff coverage is 70.37%.

@@              Coverage Diff              @@
##             develop    #2079      +/-   ##
=============================================
+ Coverage      54.44%   54.51%   +0.06%     
- Complexity      3689     3708      +19     
=============================================
  Files            570      573       +3     
  Lines          19770    19840      +70     
  Branches        1141     1144       +3     
=============================================
+ Hits           10764    10815      +51     
- Misses          8413     8430      +17     
- Partials         593      595       +2     

Impacted Files	Coverage Δ	Complexity Δ
...ever/client/replay/LocalReplayVaultController.java	54.34% <0.00%> (-1.21%)	7.00 <0.00> (ø)
...ver/client/replay/OnlineReplayVaultController.java	82.47% <0.00%> (-0.86%)	27.00 <0.00> (ø)
...ain/java/com/faforever/client/theme/UiService.java	45.07% <0.00%> (-0.43%)	24.00 <0.00> (ø)
...a/com/faforever/client/map/MapVaultController.java	58.76% <33.33%> (-1.68%)	19.00 <0.00> (ø)
...a/com/faforever/client/mod/ModVaultController.java	54.05% <33.33%> (ø)	11.00 <0.00> (ø)
...lient/map/management/MapsManagementController.java	75.75% <75.75%> (ø)	8.00 <8.00> (?)
...com/faforever/client/map/management/MapFilter.java	80.00% <80.00%> (ø)	1.00 <1.00> (?)
...ent/map/management/RemovableMapCellController.java	89.47% <89.47%> (ø)	8.00 <8.00> (?)
...main/java/com/faforever/client/map/MapService.java	68.37% <100.00%> (+0.12%)	57.00 <2.00> (+2.00)
...ava/com/faforever/client/map/UninstallMapTask.java	100.00% <100.00%> (ø)	4.00 <0.00> (ø)
... and 4 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b21de7...cd30d50. Read the comment docs.

[Marc-Spector]

"check a map" and "update a map" features will be removed from the PR due to #2114
I will do it when I have free time.

[Sheikah45]

I am also thinking that it might be better to just use a list view with highlighting rather than a table for the display since it is only for deleting maps really. What do others think?

[Marc-Spector]

Hi @Sheikah45
I agree with you. Now do not need to use the table view. I will redo the table to the list.

[Marc-Spector]

UI elements

[Sheikah45]

Looks good just a minor change

[Sheikah45]

Looks good to me. Will see if Brutus or axel have anything to say