Robin Boutros niuage

Organizations

@challengepost
@niuage
  • @niuage 77240e8
    try moving the css to platform
@niuage
  • @niuage bd583d4
    adds focused style for the bell
niuage opened pull request challengepost/reimagine#88
@niuage
Notification center
2 commits with 26 additions and 3 deletions
@niuage
@niuage
@niuage
  • @niuage 7e9b397
    twitter remarketing tag partial
@niuage
  • @niuage 765f453
    bold elements should be the same font as the content
@niuage
niuage commented on pull request challengepost/reimagine#80
@niuage

Yes, if nothing else uses .tiny, then I'd clean it up and use that :)

niuage commented on pull request challengepost/reimagine#80
@niuage

Right, I noticed that .tiny has other styles that you dont want. My main point is that it's not because 2 selectors have the same font-size value t…

niuage commented on pull request challengepost/reimagine#80
@niuage

Do you know what .tiny is used for exactly? I wouldn't use .small, that already has a definition somewhere else, to define the size here. I would p…

niuage commented on pull request challengepost/reimagine#80
@niuage

Isn't it weird that .small and .tiny (indications of size) have the same size?

niuage commented on pull request challengepost/reimagine#77
@niuage

Nice :D

niuage commented on pull request challengepost/reimagine#77
@niuage

@MGerrior A few comments, but nice job overall getting through all of this stuff... :clap:

niuage commented on pull request challengepost/reimagine#77
@niuage

:+1:

niuage commented on pull request challengepost/reimagine#77
@niuage

Same comment than above, use the mixin.

niuage commented on pull request challengepost/reimagine#77
@niuage

This can be done with a mixin, I think it's +single-transition(border-bottom-color, 0.5s, ease).

niuage commented on pull request challengepost/reimagine#77
@niuage

I haven't looked at everything yet, but before I forget: I though that holly wanted this underline on most of the links. Did the requirement change?

niuage commented on pull request challengepost/reimagine#77
@niuage

Can you use lighten here? Doesn't really matter, just weird to use it for one and not the other ;)

niuage commented on pull request challengepost/reimagine#77
@niuage

Foundation allows you to configure that with the variable $header-top-margin. (app/assets/stylesheets/reimagine2/foundation/_foundation_config.scss)

@niuage
niuage merged pull request challengepost/reimagine#78
@niuage
Devpost footer
3 commits with 65 additions and 144 deletions
@niuage
  • @niuage 3f448b0
    update all the links to point to info.devpost.com
@niuage
  • @niuage a1461a0
    update social media links
niuage opened pull request challengepost/reimagine#78
@niuage
Devpost footer
1 commit with 65 additions and 144 deletions