Fixed Issues 87, 88 (livechat on mobile devices) #162
Conversation
ghost
assigned 
There was a problem hiding this comment.
lgtm in general.
Since most changes are @media-prefixed, I don't see much impact on the rest.
@HenrikScholl For the non-@media-places, I would like to get your feedback what's the motivation and impact.
| @@ -47,7 +47,6 @@ <h1>{{title}}</h1> | |||
| {{> offlineForm (offlineData)}} | |||
| </div> | |||
| {{/if}} | |||
| {{> poweredBy }} | |||
There was a problem hiding this comment.
@HenrikScholl can you please explain why you moved the poweredBy from here to the multiple other places?
There was a problem hiding this comment.
I changed that, because I set the fix position to the Input-Form.
In the Offline-Version is no Input-Form, so the "Powered by"-Logo has the wrong Postion
| @@ -774,6 +928,8 @@ input:focus { | |||
| opacity: 0.5; | |||
| align-self: flex-end; | |||
| transition: opacity 0.15s ease-out; | |||
| bottom: 0; | |||
There was a problem hiding this comment.
@HenrikScholl what's the effect if this is not set?
Should/Could this also be limited to @media @smartphones
There was a problem hiding this comment.
This Code set a fix postion to the Chat Input-Form (Issue 88)
mrsimpson
changed the title
|
@HenrikScholl unfortunately, I only verified the
In general, it looks as if the Reverting now, please correct and create a new PR. |
|
@HenrikScholl we had to revert the PR since it affects the regular Live-Chat window. |
Livechat on mobile devices
Closes #87
closes #88