Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mp3 dom aviv yamin #4

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

AvivYamin
Copy link

No description provided.

AvivYamin and others added 17 commits September 12, 2021 21:14
… create the functions in the index. However, I haven't finished them yet.
…reates a parent song element <div> and placed it inside a songs list (row 210). inside each song there's children(title, album...). the parent of the song is the list song, the parent of the list songs is the container and the list has a sibling of a header with a h2 as a child. I could not figure out how to pass the attribute from the object (missing one argument error)
…aying song and to log the details of the song
… value by name so I attached an ID to the inputs through for loops in order to get the value of the inputs. I placed these values into the add song function to add the new song from the inputs.
…ds which button was pressed by its class name and executes the relevant function playSong/removeSong. I also played with the CSS a bit.
…ground color to blue for the duration time of that specific song.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant