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

Clean up the theme files now that header.hbs is a thing. #173

Merged
merged 1 commit into from
Jan 26, 2018

Conversation

Havvy
Copy link
Contributor

@Havvy Havvy commented Dec 13, 2017

No more having to track index.hbs between updates. \o/

There was an extra book.css in src/theme as far as I could tell.

There's no more .has_warning class, so we need to make sure to remove the first style in reference.css when we remove that warning.

No more having to track index.hbs between updates. \o/

There was an extra book.css in src/theme as far as I could tell.
@Havvy
Copy link
Contributor Author

Havvy commented Dec 13, 2017

Oh yes, this also requires that Rust is using mdbook 0.0.28 or later.

@matthewjasper
Copy link
Contributor

Looks like Rust is currently using 0.0.26, so this will need an upgrade there first. Otherwise this lgtm, but maybe someone more familiar with mdbook should have a look.

https://github.com/rust-lang/rust/blob/0b90e4e8cd068910f604f3e1fb5d03cc01f1658f/src/tools/rustbook/Cargo.toml#L11

@steveklabnik steveklabnik merged commit f74a2a6 into rust-lang:master Jan 26, 2018
@steveklabnik
Copy link
Member

merged this in manually; doing the update to 0.1 now!

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.

3 participants