Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Fix voting tab after migration differences - Close #622 #651

Merged

Conversation

yasharAyari
Copy link
Contributor

closes #622

@@ -50,6 +50,9 @@
line-height: 24px;
margin-left: 6px;
}
& :global .disable{
Copy link
Contributor

Choose a reason for hiding this comment

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

Why :global? Cannot it be used as styles.disable instead of 'disable'?

@@ -50,6 +50,9 @@
line-height: 24px;
margin-left: 6px;
}
& :global .disable{
color: rgba(0,0,0,0.38);
Copy link
Contributor

Choose a reason for hiding this comment

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

.disable also needs cursor: default;

/* react toolbar overwroght */
.input{
margin-top: -15px;
margin-top: -10px;
padding-bottom: 5px;
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO it looks better without this padding-bottom override. (I like more the default of 20px)

reyraa
reyraa previously approved these changes Aug 25, 2017
@yasharAyari yasharAyari merged commit 4fadf2d into development Aug 25, 2017
@yasharAyari yasharAyari deleted the 622-fix-voting-tab-after-migration-differences branch August 25, 2017 15:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix voting tab after-migration differences
4 participants