Skip to content

ITP May 25 | Houssam Lahlah | Codealong number facts - #43

Open
HoussamLh wants to merge 3 commits into
CodeYourFuture:mainfrom
HoussamLh:codealong-number-facts
Open

HoussamLh wants to merge 3 commits into
CodeYourFuture:mainfrom
HoussamLh:codealong-number-facts

Conversation

@HoussamLh

Copy link
Copy Markdown

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

  • Added index.html with basic structure for Number Facts app.

  • Created style.css for minimal styling and centered layout.

  • Implemented script.js to fetch and display number facts from the Numbers API.

  • Added input validation to show facts only when a number is entered.

Questions

  • Is the UI styling acceptable, or should it match exactly like in the video color/theme?

@HoussamLh HoussamLh added the 👀 Review Requirements Changes requested to meet requirements label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Review Requirements Changes requested to meet requirements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant