Ну 2 функции по сути делают одно и то же, только на разных элементах. Можно просто ссылать оба елемента на 1 функцию, а вот с таргетом наверное отдельно придется делать, но лучше тоже на эту 1 функцию сослать чтобы если поправить решишь то просто в 1 месте было
где функции "CloseModalBtn()" она закрывает через кнопку, а "CloseModal" через крестик иконки. Хотел сделать так чтобы через if была возможность, закрывать крестиком и кнопкой, в одной функции.