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

Modified bg to purple #1

Merged
merged 2 commits into from
May 30, 2024
Merged

Modified bg to purple #1

merged 2 commits into from
May 30, 2024

Conversation

brytebee
Copy link
Owner

Hello partner, 👋

In the branch/PR I have accomplished the following:

  • Change background
  • thing 1
  • thing 2
  • thing 3

Copy link
Collaborator

@olaishola05 olaishola05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brytebee You have done a wonderful job with the bg color

However, you did not use the right background color. Kindly check the inline comment to see required changes.

Happy coding!

styles.css Outdated
background-color: aquamarine;
background-color: #a0a;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brytebee this is not the right brand color, kindly check the design to adjust to the right color.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@olaishola05 I have made the suggested changes.

Copy link
Collaborator

@olaishola05 olaishola05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brytebee You have done an amazing job with the changes requested. There is nothing else to do other than merge! 👍🏿

@olaishola05 olaishola05 merged commit 422d512 into develop May 30, 2024
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.

None yet

2 participants