search

MayOneUS / pledgeservice

pledge.mayday.us
pledge.mayday.us
Apache License 2.0
11 stars 11 forks source link

More Jade-ification and cleanup #38

Closed dklann closed 10 years ago

dklann commented 10 years ago

Dunno if these will make it to production, but they will be a little cleaner if they do. They all look reasonable on localhost:8080 ~d

brad commented 10 years ago

Thanks for removing those stray p tags. Looks like there's a few more in more.jade and learnmore.jade

brad commented 10 years ago

I just have one more tiny request @dklann. In aboutus.jade you changed a link from absolute to relative. Can you please change it back? We made all the links absolute on purpose so that if people manage to find their way here, they will be pushed back to mayone.us if they click any links.

dklann commented 10 years ago

Was that the link to 'privacy-policy' ? The copy I edited was missing the protocol specifier (http[s]:). I'll simply add that if that's what you're talking about. Thanks!

brad commented 10 years ago

That's the one, but the missing protocol specifier is intentional, so that linked page uses the same protocol as the page the user was on.

dklann commented 10 years ago

Holy crap! That's a convenient feature I wasn't previously aware of! Nice. I'll put it back the way it was. Coming up ...

brad commented 10 years ago

Thanks!