Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add transitions for hover over elements #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jadx2
Copy link
Owner

@jadx2 jadx2 commented Dec 11, 2020

For this branch I used hover transitions in 3 components:

  • The subscribe button in the navbar has a hover that changes the background color to the red-danger one.
  • The title links in the top story, featured stories, and sponsored insight have a hover transition animated underline.
  • The last ones are the buttons with class btn-danger, used for the forms and subscribe section, have a hover with an increased scale.
    All of these animations are just for the desktop version of the webpage.

@jpdf00
Copy link
Collaborator

jpdf00 commented Dec 11, 2020

Used all the lessons on the code.
descriptive and professional pull request.
No linters errors.
Meaningful commits.

Excellent and professional work.

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.

2 participants