-
Notifications
You must be signed in to change notification settings - Fork 2.1k
fix(demos): Too much space between toolbar title and left icon. #2174
Changes from 23 commits
2c21d8b
35fcd55
83f71f3
a269519
9f862de
e113916
fa66363
de14f7e
e23fc01
0b33ac1
777dd7d
a779f84
e16ac8b
2e6f40e
8b62459
c57b06b
00ba0b3
98e30f2
6ab4989
ff9d773
8461d65
32b4c3d
9a82489
d12bcbb
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -79,7 +79,6 @@ fieldset { | |
|
||
.catalog-logo, | ||
.catalog-back { | ||
padding-right: 24px; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should keep the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think we should... There is too much space between the logo and the title... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah wait, there is already a There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yup, not needed anymore. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's why I removed it 😃 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yup, no longer needed. |
||
color: #eee; | ||
text-decoration: none; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn't be removing this span to change the
padding-right
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not removing the
span
? I kept the class 😃There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it's not needed anymore :)