psergus / ngWYSIWYG

true angular WYSIWYG
MIT License
63 stars 42 forks source link

a lot of buttons won't work when screen width is narrow #4

Closed butaixianran closed 8 years ago

butaixianran commented 9 years ago

A lot of buttons will leave the toolbar and won't work any more, when screen width is narrow

psergus commented 9 years ago

butaixianran, can you tell pls what width did you try?

Sent from my iPhone

On Apr 25, 2015, at 12:31 AM, butaixianran notifications@github.com wrote:

A lot of buttons will leave the toolbar and won't work any more, when screen width is narrow

— Reply to this email directly or view it on GitHub.

butaixianran commented 9 years ago

I narrow my chrome 42 to mobile size. Here is a screenshot:

2015-04-26 6-25-35

I also find a solution for that, we can update the template, add flex to toolbar's style, then it will be fine. But the gray image background will be mess, so I just remove it and use Bootstrap's class.

Check this: http://plnkr.co/edit/Doz2qGP3dBljPUe4F23x?p=preview

2015-04-26 7-19-00

psergus commented 9 years ago

good idea, let me play with it

On Sat, Apr 25, 2015 at 4:17 PM, butaixianran notifications@github.com wrote:

I narrow my chrome 42 to mobile size. Here is a screenshot:

[image: 2015-04-26 6-25-35] https://cloud.githubusercontent.com/assets/5558722/7335141/3255fd20-ebdd-11e4-9c83-91770f245645.jpg

I also find a solution for that, we can update the template, add flex to toolbar's style, then it will be fine. But the gray image background will be mess, so I just remove it and use Bootstrap's class.

Check this: http://plnkr.co/edit/Doz2qGP3dBljPUe4F23x?p=preview

— Reply to this email directly or view it on GitHub https://github.com/psergus/ngWYSIWYG/issues/4#issuecomment-96290980.

Best regards, Sergey Petrenko

psergus commented 9 years ago

I updated CSS and regrouped some buttons so they are more responsive now to the screen size change. Thx for the notice :+1:

brunokrebs commented 8 years ago

I just tested it and your fix is working just fine @psergus. If @butaixianran has nothing new to add to this issue, I think it is a good time to close it.

psergus commented 8 years ago

closing it. thx @brunokrebs and @butaixianran