oxen-io / session-android

A private messenger for Android.
https://getsession.org
GNU General Public License v3.0
1.73k stars 165 forks source link

Cannot add new contact on Android (on Chromebook) #1135

Open tiyb opened 1 year ago

tiyb commented 1 year ago

Code of conduct

Describe the bug

Using the Android version of Session on a Chromebook, there is no ability to add a new contact (or join a community) via Session ID.

To reproduce

Steps to reproduce the behavior:

  1. Go to the session main screen, and click the + Button
  2. On the New Conversation screen tap New Message or Join Community
  3. Paste the Session ID
  4. Nothing happens

Screenshots or logs

Navigate to Join Community screen:

Screenshot 2023-03-20 11 23 53 AM

Paste Session ID to Session Community:

Screenshot 2023-03-20 11 24 29 AM

Nothing happens; just a text box with the Session ID, but no button to click, no UI to indicate how to move forward.

Smartphone (please complete the following information):

Additional context

tiyb commented 1 year ago

Update: I installed the Linux version of the app on the same Chromebook, and there is a Next button in the UI, which becomes enabled when a Session ID is entered.

Screenshot 2023-03-20 2 00 07 PM

hjubb commented 1 year ago

Just as a side note, the "join community" page is for joining an open group / community via a URL, the session ID should only be entered in the "New Message", although there should also be confirmation buttons which look like they're being cut off...

tiyb commented 1 year ago

Yeah, the point is the lack of a "Next" button when trying to use the Android app on a Chromebook. I used the Community screen because I had a Session ID handy (that I was comfortable to share), but the UI is missing the button in the New Message screen as well.

hjubb commented 1 year ago

Thanks for more info. I'm not sure if this is a chromebook specific issue, the UI for that new conversation screen seems different but it might just be due to the resolution I'm not too sure. we can probably move the next button to below the input as opposed to offset from the bottom (might fix it from not displaying properly that way)

tiyb commented 1 year ago

Don't know if this is on a roadmap, but it would be great if the change could be made. At this point I have to install the Linux subsystem just to use the app on a Chromebook, which is sub-optimal.

But it also feels like the kind of visual change that will work properly on any device, once it's made.