fix ui elements overriding

src/frontend/UiDocument.cpp

@@ -45,7 +45,7 @@ std::shared_ptr<gui::UINode> UiDocument::get(const std::string& id) const {
     if (found == map.end()) {
         return nullptr;
     }
-    return found->second;
+    return found->second.lock();
 }
 
 const uidocscript& UiDocument::getScript() const {

src/frontend/UiDocument.hpp

@@ -19,7 +19,7 @@ struct uidocscript {
     bool onclose : 1;
 };
 
-using UINodesMap = std::unordered_map<std::string, std::shared_ptr<gui::UINode>>;
+using UINodesMap = std::unordered_map<std::string, std::weak_ptr<gui::UINode>>;
 
 class UiDocument {
     std::string id;

src/graphics/ui/elements/UINode.cpp

@@ -385,12 +385,15 @@ bool UINode::isSubnodeOf(const UINode* node) {
 
 void UINode::getIndices(
     const std::shared_ptr<UINode>& node,
-    std::unordered_map<std::string, std::shared_ptr<UINode>>& map
+    std::unordered_map<std::string, std::weak_ptr<UINode>>& map
 ) {
     const std::string& id = node->getId();
     if (!id.empty()) {
+        const auto& found = map.find(id);
+        if (found == map.end() || found->second.expired()) {
             map[id] = node;
         }
+    }
     auto container = std::dynamic_pointer_cast<gui::Container>(node);
     if (container) {
         for (const auto& subnode : container->getNodes()) {

src/graphics/ui/elements/UINode.hpp

@@ -289,7 +289,7 @@ namespace gui {
         /// @brief collect all nodes having id
         static void getIndices(
             const std::shared_ptr<UINode>& node,
-            std::unordered_map<std::string, std::shared_ptr<UINode>>& map
+            std::unordered_map<std::string, std::weak_ptr<UINode>>& map
         );
 
         static std::shared_ptr<UINode> find(