Skip to content

Conversation

@zakingslayerv22
Copy link
Contributor

@zakingslayerv22 zakingslayerv22 commented Jun 10, 2025

Before reading the MDN article - https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_transitions/Using_CSS_transitions as attached in the assignments, I spent a lot of time trying to figure out how the brower automatically knows that it should trigger the transition on hover.

I think informing the readers that transition works with CSS property values that change, can be more helpful and better prepare them for MDN.

Because

This PR

Issue

Closes #XXXXX

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

Before reading Josh Comeau's blog https://www.joshwcomeau.com/animation/css-transitions/ as attached in the assignments. I spent a lot of time trying to figure out how the brower automatically knows that it should trigger the transition on hover. 

I think informing the readers that transition works with CSS property values that change, can be more helpful and better prepare them for Josh Comeau's blog post.
@github-actions github-actions bot added the Content: Advanced HTML/CSS Involves the Advanced HTML/CSS course label Jun 10, 2025
@wise-king-sullyman wise-king-sullyman requested review from a team and codyloyd and removed request for a team June 26, 2025 22:21
@mao-sz
Copy link
Contributor

mao-sz commented Jan 12, 2026

Thanks for the suggestion and sorry this slipped through the cracks for so long.

I'm actually going to close this PR because while there's nothing wrong with what you've written, what you've added is already covered in the lesson, so this change just adds text that repeats exactly what's just been explained.

  • The transition shorthand doesn't need breaking down because it's introduced only after going through the separate transition-* properties.
  • The hovering action triggering the background-color value change due to the :hover pseudoclass is already covered on line 27.

@mao-sz mao-sz closed this Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content: Advanced HTML/CSS Involves the Advanced HTML/CSS course

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants