Skip to content

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

Open
dflemstr wants to merge 3 commits into
masterfrom
aligned-about-links
Open

Fix social links breaking over multiple lines like "(icon)\n@dflemstr"#140
dflemstr wants to merge 3 commits into
masterfrom
aligned-about-links

Conversation

@dflemstr

Copy link
Copy Markdown
Member

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

@hlilje

hlilje commented May 15, 2014

Copy link
Copy Markdown
Contributor

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

@michaelhakansson

Copy link
Copy Markdown
Contributor

I can change my username on Twitter and GitHub ;)

@dflemstr

Copy link
Copy Markdown
Member Author

Alternate method used now.

@michaelhakansson

Copy link
Copy Markdown
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
Copy Markdown
Member Author

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

@michaelhakansson

Copy link
Copy Markdown
Contributor

Ok. Then let's update version and merge.

@dflemstr

Copy link
Copy Markdown
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
Copy Markdown
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

hlilje commented Jun 12, 2014

Copy link
Copy Markdown
Contributor

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