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

Updates styles #1276

Merged
merged 6 commits into from
Dec 27, 2017
Merged

Updates styles #1276

merged 6 commits into from
Dec 27, 2017

Conversation

Sekhmet
Copy link
Contributor

@Sekhmet Sekhmet commented Dec 27, 2017

Changes:

  • Update text styles (username + post).
  • Update logo styles.
  • Add shadow to Topnav.

@Sekhmet Sekhmet added the style label Dec 27, 2017
@bonustrack bonustrack temporarily deployed to busy-master-pr-1276 December 27, 2017 20:51 Inactive
@@ -2,9 +2,11 @@

// Start with assigning color names to specific hex values.
@white: #fff;
@white-shadow: rgba(0, 0, 0, 0.11);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why 0.11? I prefer 0.05 did you tried?

@@ -2,6 +2,8 @@

.feed-layout {
position: relative;
min-height: 100%;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this supposed to fix the view being cut when feed page is loading? It doesn't seem to work for me

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure how to solve this one. I think JM might know how to do this.

Copy link
Contributor

Choose a reason for hiding this comment

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

This fix doesnt need to happen in this PR

height: 56px;
line-height: 56px;

&__brand {
font-size: 28px;
font-size: 32px;
Copy link
Contributor

Choose a reason for hiding this comment

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

30px is enougth

@Sekhmet Sekhmet merged commit dc232df into master Dec 27, 2017
@Sekhmet Sekhmet deleted the sekhmet-style-update branch December 27, 2017 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants