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

Use modern CSS for styling instead of background images #124

Merged
merged 4 commits into from Dec 28, 2019

Conversation

@LordAro
Copy link
Member

@LordAro LordAro commented Dec 28, 2019

After:

Background of openttd-logo.png is rather crudely removed - doesn't stand up well on a non-dark background, but it's probably good enough

LordAro added 3 commits Dec 28, 2019
They might have been useful at some point, but the css for them has been identical for some time now
@LordAro LordAro force-pushed the LordAro:nav-cleanup branch from eec0f03 to c38a9d0 Dec 28, 2019
@LordAro
Copy link
Member Author

@LordAro LordAro commented Dec 28, 2019

I was able to replicate the stripes with some CSS magic:

vendor is the most likely place for someone to have put local gems.
If this is the case, jekyll will try to compile its own site template,
which is located in something like
vendor/cache/gems/jekyll-3.2.1/lib/site_template/_posts/0000-00-00-welcome-to-jekyll.markdown.erb,
which will then fail due to an invalid strftime.

See jekyll/jekyll#5267 for further details.
@LordAro LordAro force-pushed the LordAro:nav-cleanup branch from c38a9d0 to 9370c35 Dec 28, 2019
@TrueBrain
Copy link
Member

@TrueBrain TrueBrain commented Dec 28, 2019

LGTM .. like I know anything about this stuff :D

@LordAro LordAro merged commit 787d526 into OpenTTD:master Dec 28, 2019
2 checks passed
2 checks passed
Docker build
Details
website Build #20191228.16 succeeded
Details
@LordAro LordAro deleted the LordAro:nav-cleanup branch Dec 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.