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

Fix social links breaking over multiple lines like "(icon)\n@dflemstr" #140

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dflemstr
Copy link
Member

Poor @michaelhakansson gets his links messed up like this:
bla
This fixes that problem.

@hlilje
Copy link
Contributor

hlilje commented May 15, 2014

The GitHub icon is still on line 1 on a 1366 x 768 screen in Chrome/Ubuntu.

@michaelhakansson
Copy link
Contributor

I can change my username on Twitter and GitHub ;)

@dflemstr
Copy link
Member Author

Alternate method used now.

@michaelhakansson
Copy link
Contributor

The layout goes berzerk on my phone in landscape mode (and in a certain width of the browser)
skarmavbild 2014-05-15 kl 13 19 48

@dflemstr
Copy link
Member Author

Hm... maybe let's just not fix this? :)

@michaelhakansson
Copy link
Contributor

Ok. Then let's update version and merge.

@dflemstr
Copy link
Member Author

No I meant as in closing this change and letting the problem exist....

@dflemstr dflemstr closed this May 15, 2014
@dflemstr dflemstr deleted the aligned-about-links branch May 15, 2014 13:30
@michaelhakansson
Copy link
Contributor

I just realized that the problem i mentioned already exists in the current version. So it might be better to go ahead and make these changes, to make it look better most of the time.

@dflemstr dflemstr restored the aligned-about-links branch May 15, 2014 22:09
@dflemstr dflemstr reopened this May 15, 2014
@hlilje
Copy link
Contributor

hlilje commented Jun 12, 2014

So.. should we merge this?

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