Skip to content

Fix some margin in header and paypall button#12

Merged
davidchambers merged 5 commits into
gh-pagesfrom
margin-fixes
Sep 28, 2016
Merged

Fix some margin in header and paypall button#12
davidchambers merged 5 commits into
gh-pagesfrom
margin-fixes

Conversation

@Malwine

@Malwine Malwine commented Sep 12, 2016

Copy link
Copy Markdown
Member

Fixes #11
and
fixes some spacing in the header
and
fixes #10

@Malwine Malwine changed the title Fix some marhin in header and paypall button Fix some margin in header and paypall button Sep 12, 2016
@davidchambers

Copy link
Copy Markdown
Contributor

This isn't looking quite right:

PayPal button

@Malwine

Malwine commented Sep 24, 2016

Copy link
Copy Markdown
Member Author

@davidchambers ups... what happened there... hm.... will have a look

@Malwine

Malwine commented Sep 27, 2016

Copy link
Copy Markdown
Member Author

@davidchambers when do you get that? I cant reproduce.

@Malwine

Malwine commented Sep 27, 2016

Copy link
Copy Markdown
Member Author

@davidchambers ah, in Chrome :O

@davidchambers

Copy link
Copy Markdown
Contributor

It's frustrating when things render differently in different browsers. 😠

This line seems to be the problem:

#paypal-button {
  float: right;
}

If I disable this rule the problem goes away. I can't explain this, but CSS is often inexplicable. 😝

@Malwine

Malwine commented Sep 27, 2016

Copy link
Copy Markdown
Member Author

Ah, interesting. How can we stick it to the right without float then?

@Malwine

Malwine commented Sep 27, 2016

Copy link
Copy Markdown
Member Author

@davidchambers Should be alright now. Could you check? 😊

@davidchambers

Copy link
Copy Markdown
Contributor

Could you check?

I'd like to, but I'm having trouble getting the site running.

$ bundle install
-bash: bundle: command not found

👎

$ gem install bundler
Fetching: bundler-1.13.1.gem (100%)
Successfully installed bundler-1.13.1
Parsing documentation for bundler-1.13.1
Installing ri documentation for bundler-1.13.1
Done installing documentation for bundler after 6 seconds
1 gem installed

👍

$ bundle install
...
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
...
An error occurred while installing nokogiri (1.6.8), and Bundler cannot continue.
Make sure that `gem install nokogiri -v '1.6.8'` succeeds before bundling.

👎

$ gem install nokogiri -v '1.6.8'
Building native extensions.  This could take a while...
...
To see why this extension failed to compile, please check the mkmf.log which can be found here:

  /Users/dc/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/extensions/x86_64-darwin-15/2.3.0-static/nokogiri-1.6.8/mkmf.log

extconf failed, exit code 1

👎

Ruby, why must our relationship be this way? 😢

@davidchambers

Copy link
Copy Markdown
Contributor

With @Malwine's help I got the site running. I did three things:

  1. Launched Xcode and installed additional components when prompted.
  2. Ran bundle config build.nokogiri --use-system-libraries.
  3. Ran bundle install.

I'm not sure whether step 1 was necessary, but it certainly didn't hurt. :)


For the record I'm running OS X 10.11.6, MacPorts 2.3.4, and rbenv 0.3.0-362-g0c7224a.

@davidchambers

Copy link
Copy Markdown
Contributor

The fix you made is working for me, @Malwine. :)

I noticed one more thing: the border is missing. This change should fix it:

--- a/css/main.css
+++ b/css/main.css
@@ -150,7 +150,7 @@ img {
   border-radius: 6px;
   font-size: 13px;
   background: #fec348 none repeat scroll 0 0;
-  border-color: #feb215;
+  border: 1px solid #feb215;
   color: #002f86;
 }

@Malwine

Malwine commented Sep 28, 2016

Copy link
Copy Markdown
Member Author

Thanks @davidchambers you are the pro for github markdown formatting man 😅
I adjusted it now, can we merge then? ✨

@davidchambers

Copy link
Copy Markdown
Contributor

🌳

@davidchambers davidchambers merged commit 54345d4 into gh-pages Sep 28, 2016
@davidchambers davidchambers deleted the margin-fixes branch September 28, 2016 11:35
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.

2 participants