Skip to content

London | May-2025 | Sisay Mehari | Reading List #608

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sisu860
Copy link

@Sisu860 Sisu860 commented Jul 14, 2025

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

📚 Reading List App
This pull request adds a dynamic reading list that:

Displays a list of books with title, author, and cover image

Visually distinguishes between books that have been read and unread

✅ What’s included
HTML scaffold with a #reading-list inside #content

JavaScript logic that:

Loops through the books array

Creates a list item (

  • ) for each book

    Adds

    elements for title and author

    Adds an with the book cover

    Applies green/red background based on alreadyRead value

    CSS styling to highlight read/unread books

    🧪 Testing
    All tests pass (npx jest script.test.js ✅)

    Tests cover:

    Rendering title and author

    Including an image for each book

    Correct background color based on reading status

    Test file: script.test.js

    Questions:
    None at the moment.

  • @Sisu860 Sisu860 added the Needs Review Participant to add when requesting review label Jul 14, 2025
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Needs Review Participant to add when requesting review
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant