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

Long tags break the layout #3899

Closed
dolster opened this issue Jan 26, 2013 · 8 comments
Closed

Long tags break the layout #3899

dolster opened this issue Jan 26, 2013 · 8 comments

Comments

@dolster
Copy link

dolster commented Jan 26, 2013

http://i.imgur.com/imxO7Su.jpg

@jhass
Copy link
Member

jhass commented Jan 26, 2013

Hmm, not sure what's the best way to fix it. Any ideas?

@dolster
Copy link
Author

dolster commented Jan 26, 2013

No idea right now...

Let's see if someone creative sheds a light.

@desyncr
Copy link
Contributor

desyncr commented Jan 26, 2013

I think applying the same idea as with long posts ("show more") should be enough:

diaspora-bug-ui-long-tags

@rsofaer
Copy link
Member

rsofaer commented Jan 26, 2013

I think that's quite an attractive solution.

On Sat, Jan 26, 2013 at 6:53 PM, Asphyxia [email protected] wrote:

I think applying the same idea as with long posts ("show more") should be
enough:

[image: diaspora-bug-ui-long-tags]https://f.cloud.github.com/assets/1857707/100440/5ce7ae4e-6813-11e2-913b-a0248218b5f6.png


Reply to this email directly or view it on GitHubhttps://github.com//issues/3899#issuecomment-12744753.

@DeadSuperHero
Copy link
Member

I like it. :)

On Saturday, January 26, 2013, Raphael Sofaer wrote:

I think that's quite an attractive solution.

On Sat, Jan 26, 2013 at 6:53 PM, Asphyxia <[email protected]<javascript:_e({}, 'cvml', '[email protected]');>>
wrote:

I think applying the same idea as with long posts ("show more") should
be
enough:

[image: diaspora-bug-ui-long-tags]<
https://f.cloud.github.com/assets/1857707/100440/5ce7ae4e-6813-11e2-913b-a0248218b5f6.png>


Reply to this email directly or view it on GitHub<
https://github.com/diaspora/diaspora/issues/3899#issuecomment-12744753>.


Reply to this email directly or view it on GitHubhttps://github.com//issues/3899#issuecomment-12744777.

@dolster
Copy link
Author

dolster commented Jan 27, 2013

Elegant solution.
Has my support.

@movilla
Copy link
Contributor

movilla commented Jan 29, 2013

Mmmmm with word-wrap: break-word;

Captura de pantalla de 2013-01-29 22:21:00

Captura de pantalla de 2013-01-29 22:21:14

afrerich added a commit to afrerich/diaspora that referenced this issue Mar 2, 2013
jhass added a commit that referenced this issue Mar 2, 2013
@fabianrbz
Copy link
Contributor

Closing this one, fixed in #4006

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants