hi Im working on my memory game I got a few thing to fix i need help

Hi Jose, first of all nice project! Projects like this help to improve problem solving in code a lot. By myself I created a memory game in different versions and progress on my coding experience, see following. Maybe they can be an inspiration to your project.

http://noah.teamdigitalcreative.com/

http://torbenkorb.github.io/memory-game/

Back to your problem. The best would be to create a copy of your initial array on each start of the game and shuffle this new array. There are several methods to achieve this. One of the elegant approaches I found to be something like this:

// Take an array as parameter
// create a new array of arrays with the structure [random no, value]
// sort the array by the first entry (which is the random no)
// output a new array with just the sorted values
const shuffleArray = arr => arr
    .map(item => [Math.random(), item])
    .sort((a, b) => a[0] - b[0])
    .map(list => list[1]);

Then you could initialize your game like this:

function init() {
    //Cards
    const cards = shuffleArray(deck);
    for (let i = 0; i < deck.length; i++) {
        const card = document.createElement("li");
        card.classList.add("card");
        card.innerHTML = `<i class="${card[i]}"></i>`;
        cardsContainer.appendChild(card);

        //Add Click Event
        click(card);
    }
}

The modal box for the end of the game could be created as hidden screen. At the end of the game instead of alerting you could display this screen. Simply style your box with CSS and set it to display: none. Instead of alert set the display property of this box to 'block'. Example:

document.querySelector('.YOUR_BOX_CLASS').style.display = 'block'

Hope this helps, good luck!