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

Fixed lack of blue background color in useR chapter-slide #328

Merged
merged 6 commits into from
Jun 24, 2021
Merged

Fixed lack of blue background color in useR chapter-slide #328

merged 6 commits into from
Jun 24, 2021

Conversation

josschavezf
Copy link
Contributor

To contribute to this repo, please make sure to:

  • Sign the CLA (contributor license agreement). The CLA assistant will remind you below if you have not signed the agreement before.
  • Add a news item to NEWS.md and your name to DESCRIPTION (by alphabetical order) unless you have already been listed there.
  • Provide a URL to a live example (and even better, a couple of screenshots) if you are contributing a new theme.

Thank you!

@josschavezf josschavezf changed the title Hi Yihui, after fixing the issue #302 the chapter-slide lost its blue background color, I fixed this problem in this pull-request. I also replaced the Dropbox link of the useR logo with the official web page link, thanks to your recommendation when I created the theme. Fixed lack of blue background color in useR chapter-slide Jun 24, 2021
@josschavezf
Copy link
Contributor Author

josschavezf commented Jun 24, 2021 •
edited
Loading

Hi Yihui, after fixing issue #327 the chapter slide lost its blue background color, I fixed this problem in this pull request. I also replaced the Dropbox link of the useR logo with the official web page link, thanks to your recommendation when I created the theme.

yihui
yihui approved these changes Jun 24, 2021
View reviewed changes
Copy link
Owner

@yihui yihui left a comment

Choose a reason for hiding this comment

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

Ah, too bad that I made the CRAN release too soon after the previous fix... Thank you very much!

@yihui yihui merged commit cea0773 into yihui:master Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yihui yihui yihui approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@josschavezf @yihui