Skip to content

Barcelona | Oct2025-2 | Roman Pavlenko | Sprint-3 | Quote generator#4

Open
DraftRoman wants to merge 2 commits intomainfrom
quote-generator
Open

Barcelona | Oct2025-2 | Roman Pavlenko | Sprint-3 | Quote generator#4
DraftRoman wants to merge 2 commits intomainfrom
quote-generator

Conversation

@DraftRoman
Copy link
Owner

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

Changelist

  • When the page loads, it should show a random quote from the quotes array on the screen.
  • It should also show who said the quote.
  • When you click a button on the screen, it should change the quote on the screen.
  • It can look however we like, but there is an example in this folder at quote_generator_example.png

Stretch

The next steps

  • Add auto-play to your quote generator
    For this task, we will return to your quote generator and improve it with "auto-generate" functionality:
  • Add a toggle-switch or checkbox input to your existing quote generator.
  • When the switch is ON:
    ... the generator should pick and display a new quote every 60 seconds. (When testing, we will probably want to reduce this to every 5 seconds.)
    ... the generator should display "auto-play: ON" somewhere on the page.
  • When the switch is OFF
    ... the generator should NOT change the quote automatically

Questions

Ask any questions you have for your reviewer.

- split the data (file quotes.js) and code (code.js)
HTML
- added <script type="module" src="code.js"></script> line
- used exports and imports
-the function new Quote passes the quote and the author  to the DOM every time button is clicked
- style.css have been attached with basic style
- The HTML file matches the CSS file.
- The Javascript added event listener DOMContentLoaded, we have our first quote straightaway.
@DraftRoman
Copy link
Owner Author

Screenshot 2026-01-28 121045

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