julien-nc / cospend-nc

💰 💲hared budget manager Nextcloud app
GNU Affero General Public License v3.0
213 stars 29 forks source link

Keyboard on mobile Devices does not work on latest app #248

Open GAS85 opened 6 months ago

GAS85 commented 6 months ago

Hallo, I did update to the 1.5.14 and suddenly onscreen keyboard stop working on Android and iOS devices. I can't troubleshoot now from the vacation because have only mobile phone. But here attached video.

https://github.com/julien-nc/cospend-nc/assets/6813635/c538613c-470d-4886-b851-1736a36dd610

julien-nc commented 6 months ago

Hey, thanks for the bug report. I can reproduce it. No idea why this happens though. Something is preventing the focus on bill edition fields.

I found a workaround: Press the top left back arrow, open the left sidebar (top left hamburger icon next to "New Bill"), then close the left sidebar and select a bill again. Then it works fine.

Cospend's frontend does not mess with the focus. I suspect something is wrong with the @nextcloud/vue library components. This issue might be solved there at some point.

GAS85 commented 6 months ago

Cool! Workaround seems work.

Tommot8 commented 5 months ago

Have the same issue. THX so far for the workaround.

leafonthewind commented 5 months ago

One discovery: If I set Safari to Responsive Mode, widths below 500px have the problem (when the label for the text box is above the text box itself). Widths above 500px (when the field labels flow to the left-hand side, same line as the text entry boxes) do not have this problem.

First image below is the problem (text fields will not focus) using Safari responsive mode. Second image is a wider viewport allowing focus on those text fields.

Screenshot 2024-02-01 at 6 18 03 PM Screenshot 2024-02-01 at 6 18 18 PM

The workaround from @julien-nc is not working consistently (for me, at least!) running Nextcloud 28.0.2 and CoSpend 1.6.0 and using an iPhone 14 running iOS 17.3. On iOS/iPhone + Safari I can never get the keyboard to appear. On macOS + Safari running in Responsive Mode...if I change the viewport width back/forth a couple times, then I can get the text entry working when the viewport is small (phone-sized).

Fwiw, the console log has an error: "ResizeObserver loop completed with undelivered notifications." However, it does not appear with any consistency when I am reproducing the error. Might be unrelated.

lorenzo-w commented 4 months ago

Experiencing the same issue with Firefox on Android. Mine seems to be cause by opening the dialog to select a file and is fixed by the workaround consistently.

leafonthewind commented 4 months ago

I don't have an Android device to test with, but I noticed that by using Chrome browser in responsive mode (to get a narrow viewport) on a Mac desktop I could (a) trigger the bug and (b) consistently use the work-around.

Safari/WebKit is where the work-around does not seem to work.

brunothg commented 3 months ago

Still the same with 1.6.1 (Nextcloud 28.0.3). Workaround is just fine (but annoying).

adripo commented 3 months ago

Hi @julien-nc, I tried to debug this issue and found out that the fields (happens on both text fields and multiselect) are blocked from interactions on screens having width < 512 px

2024-03-21_03-35-26-007_brave

2024-03-21_03-36-06-187_brave

I searched for references in cospend's code without success, so maybe the issue is indeed in the vue libraries.

adripo commented 3 months ago

@julien-nc I found out a related issue on vue's repo: https://github.com/nextcloud-libraries/nextcloud-vue/issues/5385

When clicking on the textarea, the focus is attached to the logo link instead of the textarea when width < 512px.

width 511px:

2024-03-21_03-41-46-939_brave

width 512px:

2024-03-21_03-46-53-560_brave

TheCrimsonLady commented 3 months ago

I just updated my Nextcloud to 28.04, but the issue still persists on my iPhone 12 Pro Max with iOS 17.3.1.

What infos can I give you to help solve the problem?

42ske commented 3 months ago

The work-around does not work for me with latest Firefox and Safari on an iPhone SE with iOS 17.4.1

TheCrimsonLady commented 2 months ago

just updated to NC 29.0.0 and it still doesn't work. What infos can I provide to help solve this?

it's still the exact same behaviour as shown in the video in the first comment.

mhastu commented 3 weeks ago

Btw you also cannot create a project on the mobile web view because of that issue.