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.

Sign up for GitHub

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

Jump to bottom

adding new, blue styling! #13

Merged
merged 5 commits into from
May 1, 2021
Merged

adding new, blue styling! #13

merged 5 commits into from
May 1, 2021

Conversation

MayaGans
Copy link
Contributor

@MayaGans MayaGans commented Apr 30, 2021 •
edited
Loading

Changelog

  • center the R logo
  • change links to blue
  • update color to blue hex as per @mine-cetinkaya-rundel's suggestion
  • update little info boxes

Screen Shot 2021-04-30 at 11 54 42 AM

@MayaGans MayaGans requested a review from hturner April 30, 2021 18:56
willingc
willingc approved these changes Apr 30, 2021
View reviewed changes
Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

I love this PR and making the devguide more visually appealing. ❤️

@mine-cetinkaya-rundel
Copy link
Contributor

Agreed, looks great!

@willingc
Copy link
Contributor

willingc commented May 1, 2021

@hturner I'm going to merge this. We can always modify in a future PR if desired.

Thanks @MayaGans for the PR and @mine-cetinkaya-rundel for the review 💐

@willingc willingc merged commit 4c2ae98 into master May 1, 2021
@willingc willingc deleted the css branch May 1, 2021 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@willingc willingc willingc approved these changes

@mine-cetinkaya-rundel mine-cetinkaya-rundel mine-cetinkaya-rundel approved these changes

@hturner hturner Awaiting requested review from hturner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MayaGans @mine-cetinkaya-rundel @willingc