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

Changed text orientation #30

Merged
merged 1 commit into from
Feb 28, 2014
Merged

Changed text orientation #30

merged 1 commit into from
Feb 28, 2014

Conversation

tcfunk
Copy link
Contributor

@tcfunk tcfunk commented Feb 28, 2014

Left-alignment on .tyto-item-header h6 and .tyto-item-content looks cleaner to me than center-alignment.

Left-alignment on the tyto-item-header and tyto-item-content looks cleaner to me than the center-align.
jh3y pushed a commit that referenced this pull request Feb 28, 2014
Changed text orientation
@jh3y jh3y merged commit 9b0f676 into jh3y:master Feb 28, 2014
@jh3y
Copy link
Owner

jh3y commented Feb 28, 2014

Hey @logorrhea

Thanks for this, I wasn't too sure to be honest but now I've had a look at the changes I kinda agree.

Plus it's natural that you would write from left to right on a post it, not centre aligned.

Thanks again!

@jh3y

@tcfunk
Copy link
Contributor Author

tcfunk commented Feb 28, 2014

Glad I could help! I like what you're doing here.

@jh3y
Copy link
Owner

jh3y commented Feb 28, 2014

Thanks!

@jh3y
Copy link
Owner

jh3y commented Feb 28, 2014

You could do me one massive favour if you like ;)

If you clone the repo and run npm install do you run into any issues?

I am getting errors but I am not sure if package.json is broken or if my node is playing up.

@jh3y

@jh3y
Copy link
Owner

jh3y commented Feb 28, 2014

I reinstalled node and all was well urgh lol.

@jh3y

@tcfunk
Copy link
Contributor Author

tcfunk commented Feb 28, 2014

Mine failed on the first try too. I was running node v0.10.24.

Had to upgrade node to v0.10.26 I tried again, and it worked fine for me as well. Not sure if that was a coincidence or not!

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.

2 participants