Skip to content

Peer to peer review, Feb 24 2023 #4

@jlvFlores

Description

@jlvFlores

Great job on your to-do list, I liked your styles. Here are some suggestions to make it better:

  • On your index.js file lines 17 to 21, you can reduce the amount of lines of code by pushing and declaring your object in one line like so
    • crud.tasks.push({description: textInput.value, completed: false, index: index});
  • Be sure to remove any commented code.
  • Try to remove as many of the eslint disable comments, I understand not all are going to be possible.
  • Make sure you properly indent your lines of code, so that it may be easier to read.
  • You can declare your styles in CSS so that you aren't adding them through JavaScript
  • Make sure that the width of your textarea doesn't change when it's focused.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions