Cape Town | 2026-ITP-Jan | Isaac Abodunrin | Sprint 3 | Grouping data: Quote generator

[bytesandroses]

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Changelist

• Created a responsive quote generator webpage styled using quote_generator_example.png as a guide.
• When the page loads (or the New quotes button pressed), a quote is randomly selected and the following is displayed:
  • Quote
  • Author
  • New quote button

[cjyuan]

The leading - appears to be for styling purposes. Keeping it in the CSS or in HTML could allow us to adjust the view without changing any JavaScript code.

[bytesandroses]

You're absolutely right -- removing the - from JavaScript would be better for separation of concerns. I've moved the - into the CSS. I've also moved the leading “ in front of each quote from the HTML into the CSS so it's more consistent.

[cjyuan]

The code that setup the event listener once is also part of the "run on load" code.
You can consider including all the "run on load" code in a function to make it clearer
that "this is what runs when the page loads."

For examples,

function setup() {
  // code to be executed on page load
}

window.addEventListener('load', setup);

or

window.addEventListener('load', function() {
  // code to be executed on page load
});

[bytesandroses]

Thanks for the suggestion CJ. I've gone ahead and wrapped the code needed to run on loading into a setup function.

[cjyuan]

Looks good. Well done.