Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

If I long-press on URL bar with an externally-copied URL in my clipboard, Android spams a notification "Firefox pasted from your clipboard" which is confusing and covers up the actual "paste" option #26564

Closed
dholbert opened this issue Aug 20, 2022 · 17 comments
Labels
Android:13 Issues specific to Android 13 - API 33 🐞 bug Crashes, Something isn't working, .. Feature:ContextMenu Menu that appears when long-pressing on website content Feature:Toolbar Address bar, see also Feature:Search good first issue Good for newcomers qa-triaged Issues triaged by qa S3 Blocks non-critical functionality and a work around exists

Comments

@dholbert
Copy link
Contributor

dholbert commented Aug 20, 2022

Steps to reproduce

  1. Copy a URL from some external (non-Firefox) app. (e.g. in Chrome, visit https://www.example.org/, and then tap their URLbar and then copy-to-clipboard button.)
  2. Open Firefox Nightly.
  3. Long-press the URL bar.

Expected behaviour

Menu should appear saying "Paste | Paste & Go" (and "Copy" if there's already contents in the URLbar)

Actual behaviour

That menu appears but it's covered up by an Android system notification, saying "Firefox pasted from your clipboard".

This notification is troublesome for two reasons:
a) it blocks me (for a few seconds) from proceeding with the paste operation that I was about to do.
b) it's alarming & feels shady, like something is going on behind my back. I didn't actually paste yet, but Android's alerting me that Firefox already pasted out from under me...? (I know nothing actually-shady is going on; probably Firefox is just inspecting the clipboard to find out if it's a URL so as to offer the appropriate menu options. But this notification doesn't offer any of that subtlety. It just says "Firefox pasted from your clipboard", which feels alarming when all I did was long-press.)

Device name

Google Pixel 4a

Android version

Android 13

Firefox release type

Firefox Nightly

Firefox version

103.2.0

Device logs

No response

Additional information

  • I can reproduce in latest Firefox Nightly as well, version 105.0a1.
  • If you move the Firefox toolbar to the top, that mitigates the problem with the notification covering up our menu (since then our menu is at the top while the notification is still at the bottom). Not really a workaround, just a FYI.
  • This notification only appears the first time you long-press with a given set of clipboard-contents. So if you long-press again after performing the STR, then the notification doesn't show up again. But if you switch apps and copy a new URL and then switch back to Firefox and long-press the URLbar, you'll hit the issue again.

┆Issue is synchronized with this Jira Task

@dholbert dholbert added needs:triage Issue needs triage 🐞 bug Crashes, Something isn't working, .. labels Aug 20, 2022
@dholbert
Copy link
Contributor Author

Screencast:
https://user-images.githubusercontent.com/426803/185754040-fecb5e9e-82ae-4c81-b57e-3dfac13e5f40.mp4

Screenshot of the moment the issue happens (when I want to tap "Paste"/"Paste&Go" but they're covered up by the notification that confusingly tells me that Firefox already pasted even though I didn't ask it to yet):
screenshot-2022-08-20-081854

@mcarare
Copy link
Contributor

mcarare commented Aug 22, 2022

That is the system notification, and we do not control the content. I do agree it is confusing.

@mcarare mcarare added the Android:13 Issues specific to Android 13 - API 33 label Aug 22, 2022
@delia-pop
Copy link

Hello,
I can reproduce the issue as described with Google Pixel 6 (Android 13) on the latest Nightly from 08/22.

@delia-pop delia-pop added Feature:Toolbar Address bar, see also Feature:Search S3 Blocks non-critical functionality and a work around exists Feature:ContextMenu Menu that appears when long-pressing on website content qa-triaged Issues triaged by qa labels Aug 22, 2022
@dholbert
Copy link
Contributor Author

dholbert commented Aug 22, 2022

That is the system notification, and we do not control the content.

Yup, I know/understand. Nonetheless, we're doing something to make it appear; and we're simultaneously popping up our own menu directly behind it, with options that we know that the user probably wants to tap.

I do agree it is confusing.

FWIW for me personally, the main pain-point here isn't the confusion; it's more the usability issue.

@amedyne amedyne added Needs-UX Issues or tickets that need UX input or review good first issue Good for newcomers and removed needs:triage Issue needs triage labels Sep 1, 2022
@Timshel
Copy link

Timshel commented Sep 14, 2022

Hey, found the issue interesting had a look running current master.

Found something else when testing, if the field is empty and the clipboard contain a non url text then when trying to paste the notification will trigger but no option to paste will appear (If there is text in the url/search bar then Copy or Paste is offered). Should it be opened in it's own issue ?

The notification issue is linked to the "Paste & Go" option which require to check the content of the clipboard. Built a quick version without it and the notification appear only after the text is pasted. Additionally since it's a simple paste the keyboard open and the notification appear over it so it's less intrusive (the pasted text is still visible).

Due to the delay before the notification disappear the "Paste & Go" make navigation simpler but slower; if the option is not offered the user need to press navigate but it's still visible even when the notification is active.

I don't think moving the different actions is really possible (moving it over the urlbar would obfuscate it not really pertinent if the user want to copy ? and moving those actions anywhere else would make them disjointed ?).

So as a fix the only option i see at the moment is to remove the "Paste & Go" option or is there something else I'm missing that could be done ? 🤔

@mcarare
Copy link
Contributor

mcarare commented Sep 19, 2022

Users have the option to disable the toast message alert from the OS settings menu (Settings -> Privacy -> Show clipboard access toggle).

@lime124
Copy link
Collaborator

lime124 commented Sep 22, 2022

removing needs ux label - i'm not sure what we can do here to improve the situation without first understanding all the constraints and possibilities from eng. from what i can tell this notification is controlled by the OS and caused by the paste and go feature. i don't believe we are wanting to remove the feature. none of that makes it less frustrating though.

@Timshel
Copy link

Timshel commented Sep 24, 2022

Note that the flow while similar is worse (imo) when the intent is to copy.

After the long press the user get a vibration feedback, the options open (copy, paste ...) but is masked by the OS notification. If one just quickly glance at it they might think that the url is copied (got tricked recently 😅) and will realize it's not the case only when trying to paste it later on (why I call it worse).

Additionally while minor it means that Fenix unnecessarily read the clipboard (but no way to know the user intent).

@Timshel
Copy link

Timshel commented Oct 7, 2022

Hey,
Some feedback on the issue ?
Got "tricked" again yesterday when copying an url; between the vibrating feedback and the OS notification masking the buttons I was already switching app before I remembered that I had in fact not copied the url 😅.

Timshel pushed a commit to Timshel/fenix that referenced this issue Oct 10, 2022
…ad clipboard content and allow to paste non url
@github-actions github-actions bot added the eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged label Oct 10, 2022
@Timshel
Copy link

Timshel commented Oct 10, 2022

While discussing #27182 @Mugurell mentioned #22271 which introduced the logic to determine if we may have an url without reading the clipboard.

The logic was previously introduced but was then removed due to different bugfixes. Created a PR to revert it.

Timshel pushed a commit to Timshel/fenix that referenced this issue Oct 18, 2022
…ad clipboard content and allow to paste non url
@Timshel
Copy link

Timshel commented Oct 25, 2022

Hey,
I created a PR would be interested on some feedback on it :)

@Timshel
Copy link

Timshel commented Nov 2, 2022

Ping

@Timshel
Copy link

Timshel commented Nov 15, 2022

Pong

@onemahon
Copy link

Hey there! Sorry for the delay on this, but thanks the original report, and @Timshel for proposing a patch! I'll help find someone to help review that PR.

Timshel pushed a commit to Timshel/fenix that referenced this issue Nov 17, 2022
…ad clipboard content and allow to paste non url

Fixes

Update app/src/main/java/org/mozilla/fenix/utils/ClipboardHandler.kt

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>

Update app/src/main/java/org/mozilla/fenix/utils/ClipboardHandler.kt

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>

fix

fix
@Timshel
Copy link

Timshel commented Jan 3, 2023

Happy new year :).

Hey, PR is still blocked since changes fix another issue (#27182) for which nobody has been able to give any details/insight.

@cpeterso cpeterso changed the title [Bug]: If I long-press on URLbar with an externally-copied URL in my clipboard, Android spams a notification "Firefox pasted from your clipboard" which is confusing and covers up the actual "paste" option If I long-press on URL bar with an externally-copied URL in my clipboard, Android spams a notification "Firefox pasted from your clipboard" which is confusing and covers up the actual "paste" option Jan 12, 2023
@cpeterso
Copy link

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1810045

Change performed by the Move to Bugzilla add-on.

@github-actions github-actions bot reopened this Jan 12, 2023
@github-actions github-actions bot added eng:qa:needed QA Needed and removed eng:reopen-for-qa Reopens and tags the issue for QA needed when the issue is merged labels Jan 12, 2023
@cpeterso cpeterso removed eng:qa:needed QA Needed Needs-UX Issues or tickets that need UX input or review labels Jan 13, 2023
@cpeterso
Copy link

QA is not needed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android:13 Issues specific to Android 13 - API 33 🐞 bug Crashes, Something isn't working, .. Feature:ContextMenu Menu that appears when long-pressing on website content Feature:Toolbar Address bar, see also Feature:Search good first issue Good for newcomers qa-triaged Issues triaged by qa S3 Blocks non-critical functionality and a work around exists
Projects
None yet
8 participants