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

Chatboxes no more visible on a mobile browser or on narrow desktop window #1683

Open
deleolajide opened this issue Aug 18, 2019 · 7 comments
Labels
bug mobile UI User interface

Comments

@deleolajide
Copy link
Member

deleolajide commented Aug 18, 2019

Describe the bug

When converse is shown in a mobile browser or in a narrow desktop window, it used t be possible in 4.2 and below to show the selected chatbox by hiding the control box. Now in 5.0.2, the chatbox is not visible as the control box remains visible.

Current behavior
image

Clicking on florence does not change the screen

Expected behavior
image

Clicking on florence changes the screen

Environment (please complete the following information):

  • Desktop and Mobile
  • Converse.js version [e.g. 5.0.1]

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@deleolajide deleolajide changed the title Chatboxes no more visible on a mobile browser or desktop window is narrow Chatboxes no more visible on a mobile browser or on narrow desktop window Aug 18, 2019
@joudinet
Copy link
Contributor

I experience the same issue on the embedded view:

embedded_css_issue_on_mobile_view

Looking at the CSS, even though I'm far from an expert, I think there is something wrong with the rules related to the classes: controlbox and flyout box-flyout.

@joudinet
Copy link
Contributor

Here is my workaround to this issue:
Activate singleton option to show only one room in the embbeded view, and add the following CSS rules to fix the rendering on small screens:

@media screen and (max-width: 480px) {
    #page-content #conversejs .chatroom .box-flyout {
	height: 100% !important;
    }
}

@media screen and (max-width: 768px) {
    #conversejs .converse-chatboxes .chatbox .box-flyout {
	bottom: initial;
	margin-left: 0;
    }
}

This workaround is good enough for my use case but it is not good enough to close this ticket.

@deleolajide
Copy link
Member Author

Looks like this issue is fixed in version 5.0.5. At least this workaround is no more needed

@joudinet
Copy link
Contributor

I've just tested v5.0.5 and I still need to set the bottom property to initial and the margin-left to 0 on small screens. Did you try on small screen as well? For example by activating the Responsive Design Mode (Ctrl+Shift+M) on Firefox.
Without this workaround, the singleton embedded view sets .chatbox.box-flyout margin-left to 15px and bottom to 0, which produces the incorrect positioning reported in #1683 (comment)

@joudinet joudinet reopened this Nov 25, 2019
@deleolajide
Copy link
Member Author

Did you try on small screen as well?

image

This work-around actually stopped 5.0.5 from working for me in narrow window mode as the controlbox became invisible as soon as the screen narrowed. It took a bit of debugging before I narrowed it down to this change.

I have not however not yet tried it on my phone in PWA mode

@joudinet
Copy link
Contributor

FYI, I've just tested on version 6.0.0 and the layout of the embedded view without singleton mode is still not usable on a mobile browser or on a narrow desktop window.
The CSS workaround is still required for the embedded view with singleton mode.

@jcbrand
Copy link
Member

jcbrand commented Jan 20, 2020

@joudinet That's a different issue since you're using embedded view. Please make a new issue for it.

@Echolon Echolon added mobile UI User interface labels Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug mobile UI User interface
Projects
None yet
Development

No branches or pull requests

5 participants