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

Fix CSS #10

Merged
merged 16 commits into from Jun 11, 2020
Merged

Fix CSS #10

merged 16 commits into from Jun 11, 2020

Conversation

utkarsh2102
Copy link
Contributor

Hi @FabioRosado,

Thanks for working on this awesome project! It looks lovely!
However, there were some CSS problems as reported in #9.
This PR fixes the same 💯 🚀

Hope that helps!
Also, I committed two things separately, using the latest bundler and the fix for CSS. It'd be nice if you don't squash these commits as they're two different things 😄

Fixes: #9

dependabot-support and others added 14 commits March 25, 2019 06:20
Bumps [jekyll-feed](https://github.com/jekyll/jekyll-feed) from 0.11.0 to 0.12.1.
- [Release notes](https://github.com/jekyll/jekyll-feed/releases)
- [Changelog](https://github.com/jekyll/jekyll-feed/blob/master/History.markdown)
- [Commits](jekyll/jekyll-feed@v0.11.0...v0.12.1)

Signed-off-by: dependabot[bot] <support@dependabot.com>
…jekyll-feed-0.12.1

Bump jekyll-feed from 0.11.0 to 0.12.1
Bumps [jekyll](https://github.com/jekyll/jekyll) from 3.8.5 to 3.8.6.
- [Release notes](https://github.com/jekyll/jekyll/releases)
- [Changelog](https://github.com/jekyll/jekyll/blob/master/History.markdown)
- [Commits](jekyll/jekyll@v3.8.5...v3.8.6)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
…jekyll-3.8.6

Bump jekyll from 3.8.5 to 3.8.6
Bumps [jekyll-feed](https://github.com/jekyll/jekyll-feed) from 0.12.1 to 0.13.0.
- [Release notes](https://github.com/jekyll/jekyll-feed/releases)
- [Changelog](https://github.com/jekyll/jekyll-feed/blob/master/History.markdown)
- [Commits](jekyll/jekyll-feed@v0.12.1...v0.13.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
…jekyll-feed-0.13.0

Bump jekyll-feed from 0.12.1 to 0.13.0
Bumps [jekyll](https://github.com/jekyll/jekyll) from 3.8.6 to 3.8.7.
- [Release notes](https://github.com/jekyll/jekyll/releases)
- [Changelog](https://github.com/jekyll/jekyll/blob/master/History.markdown)
- [Commits](jekyll/jekyll@v3.8.6...v3.8.7)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
…jekyll-3.8.7

Bump jekyll from 3.8.6 to 3.8.7
@utkarsh2102
Copy link
Contributor Author

Ah, it's weird that the netlify deploy is failing -- it's because of using bundler 2.1.4.
However, the netlify builds fine for me. I am using 2.1.4 and my site is deployed on netlify itself.

Any clues?

assets/style.css Outdated Show resolved Hide resolved
Copy link
Owner

@FabioRosado FabioRosado left a comment

Choose a reason for hiding this comment

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

Hello @utkarsh2102,

Again thank you so much for taking the time to fix this issue!

If I'm not mistaken the problem with the deploying happened because you committed your Gemfile.lock, I was using an old version and that was probably why the error happened.

I'm assuming that bundling the whole thing with 2.1.4 will be fine - I should update the whole project sice I'm using an old version of Jekyll as well 😄

Anyway, I've added a suggestion for the menu just because it was breaking a bit on my 13'' screen due to lack of space and I believe using flex and justify to center should work on your end as well 🤔

utkarsh2102 and others added 2 commits June 9, 2020 18:14
@utkarsh2102
Copy link
Contributor Author

Hi @FabioRosado,

Again thank you so much for taking the time to fix this issue!

Thanks to you, too! 😄

I'm assuming that bundling the whole thing with 2.1.4 will be fine - I should update the whole project sice I'm using an old version of Jekyll as well smile

Perfect, works alright now! 💯

Anyway, I've added a suggestion for the menu just because it was breaking a bit on my 13'' screen due to lack of space and I believe using flex and justify to center should work on your end as well thinking

Yeah, applied the suggestion; good to go now, I think! :)

Copy link
Owner

@FabioRosado FabioRosado left a comment

Choose a reason for hiding this comment

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

Again thank you so much for working on this issue @utkarsh2102 - both for spotting the problem and being happy to fix it. I'm happy with these changes and going to merge the PR now 😄 👍

@FabioRosado FabioRosado merged commit 7141762 into FabioRosado:master Jun 11, 2020
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.

Some broken CSS
4 participants