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

CHE-98: Restyle check-boxes position in 'git commit' view #971

Merged
merged 1 commit into from
Apr 14, 2016
Merged

Conversation

vinokurig
Copy link
Contributor

gitcommit
@vparfonov please review

@TylerJewell
Copy link

@vinokurig - it looks like the check boxes are not vertically spaced properly?

@vinokurig
Copy link
Contributor Author

I have separated with space 'Amend previous commit' check-box because this operation differs from those that are above, @TylerJewell what do you think?

@TylerJewell
Copy link

@vinokurig - thank you for the explanation. If you hadn't said that, i would have not understood it. I think to the average user, it will look weird, though. It would need to be separated by a separator to make it clear that we mean something different. I think it's safer to keep it all evenly spaced only because it is not so intuitive.

@vinokurig
Copy link
Contributor Author

gitcommit
Aligned all checkboxes in common way

@vparfonov
Copy link
Contributor

ok

@vinokurig vinokurig merged commit 44fd6f9 into master Apr 14, 2016
@vinokurig vinokurig deleted the CHE-98 branch April 14, 2016 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants