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

[Telemetry] Text Selection Toolbar #11580

Closed
7 tasks
vesta0 opened this issue Jun 15, 2020 · 7 comments
Closed
7 tasks

[Telemetry] Text Selection Toolbar #11580

vesta0 opened this issue Jun 15, 2020 · 7 comments
Assignees
Labels
Feature:ContextMenu Menu that appears when long-pressing on website content Feature:Telemetry Feature:TextSelection P2 Upcoming release

Comments

@vesta0
Copy link
Collaborator

vesta0 commented Jun 15, 2020

Description & Product Manager / Data Scientist User Story

As a product manager, I want to know which context menu option are used frequently when users long press on text on a page, so we can ensure that the order matches our users' needs and saves them time.

What questions will you answer with this data?

-What percentage of users tab on any of the long press context menu options (copy, share, search, select all)?
-Which context menu options are used most frequently.

Acceptance Criteria

  • ENG files a DS JIRA request outlining their methodology.
  • DS sign off on instrumentation methodology addressing product questions.
  • Event pings can be queried via re:dash
  • Event pings can be queried via amplitude
  • We are sending telemetry events for the actions listed in the requirements
  • We have documented the telemetry
  • We have asked a data steward to review the telemetry

┆Issue is synchronized with this Jira Task

@vesta0 vesta0 added Feature:Telemetry P3 Some future sprint Feature:ContextMenu Menu that appears when long-pressing on website content labels Jun 15, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Jun 15, 2020
@vesta0 vesta0 removed the needs:triage Issue needs triage label Jun 15, 2020
@vesta0 vesta0 added this to Backlog in Feature engagement Jun 15, 2020
@sblatz
Copy link
Contributor

sblatz commented Jun 15, 2020

This lives in AC

To be clear this is for the TEXT SELECTION contextual menu.

Notes

  • Add this under category contextual_menu

@sblatz sblatz moved this from Backlog to Eng Ready in Feature engagement Jun 16, 2020
@vesta0 vesta0 changed the title [Telemetry] Context menu when selecting text [Telemetry] Text Selection Toolbar Jul 2, 2020
@vesta0 vesta0 added P2 Upcoming release and removed P3 Some future sprint labels Jul 2, 2020
@data-sync-user data-sync-user changed the title [Telemetry] Text Selection Toolbar FNX3-14744 ⁃ [Telemetry] Text Selection Toolbar Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-14744 ⁃ [Telemetry] Text Selection Toolbar FNX-12535 ⁃ [Telemetry] Text Selection Toolbar Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-12535 ⁃ [Telemetry] Text Selection Toolbar FNX2-13527 ⁃ [Telemetry] Text Selection Toolbar Aug 11, 2020
@kbrosnan kbrosnan changed the title FNX2-13527 ⁃ [Telemetry] Text Selection Toolbar [Telemetry] Text Selection Toolbar Aug 29, 2020
@vesta0 vesta0 added this to Telemetry in Fenix Product Backlog Sep 15, 2020
@codrut-topliceanu
Copy link
Contributor

which context menu option are used frequently when users long press on text on a page

Ok. So maybe a type: counter for each of the text selection contextual menu options. (DefaultSelectionActionDelegate.kt)

What percentage of users tap on any of the long press context menu options

Perhaps a type: boolean to know if users have ever activated any of these options before.

image

@kglazko @sblatz Does that sound/look ok to you?

@kglazko
Copy link
Contributor

kglazko commented Dec 8, 2020

Yes that looks good to me! I agree with the boolean to indicate if an option was used. I do wonder though if sequential matters at all in these interactions with this toolbar.. I will check in with Data Science and follow up.

@Mugurell
Copy link
Contributor

Talked offline with Kate who confirmed with DS and Boek:

  • The newly added metrics should log events, not counters anymore
  • An an expiration date 6 months is preferred.
    Maybe if there is already related telemetry for that part of the screen / feature we can reuse the existing expiration date so that all related telemetry probes will be renewed at the same time.

codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Dec 10, 2020
Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Dec 10, 2020
Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Dec 14, 2020
Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Jan 4, 2021
Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.
codrut-topliceanu pushed a commit to codrut-topliceanu/fenix that referenced this issue Jan 4, 2021
Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.
codrut-topliceanu added a commit that referenced this issue Jan 4, 2021
* For #11580 - Tracks text selection context menu usage

Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>
@codrut-topliceanu codrut-topliceanu self-assigned this Jan 4, 2021
@lobontiumira
Copy link

Verified on the 1/5 Nightly build using a Google Pixel (Android 10):

  1. contextual menu: copy_tapped ✔️
    Ping a1c8a219-1dcf-41e0-afce-32d106f1097a
{
          "category": "contextual_menu",
          "name": "copy_tapped",
          "timestamp": 10379
        }
  1. contextual menu: search_tapped ✔️
    Ping a1c8a219-1dcf-41e0-afce-32d106f1097a
{
         "category": "contextual_menu",
         "name": "search_tapped",
         "timestamp": 18838
       }
  1. contextual menu: select_all_tapped ✔️
    Ping a1c8a219-1dcf-41e0-afce-32d106f1097a
{
          "category": "contextual_menu",
          "name": "select_all_tapped",
          "timestamp": 22404
        }
  1. contextual menu: share_tapped ✔️
    Ping a1c8a219-1dcf-41e0-afce-32d106f1097a
{
          "category": "contextual_menu",
          "name": "share_tapped",
          "timestamp": 29625
        }

Logcat
Glean dashboard

@codrut-topliceanu there is no "contextual menu: long_press_tapped" event generated.
Could you please review?

@lobontiumira lobontiumira removed the eng:qa:needed QA Needed label Jan 5, 2021
@codrut-topliceanu
Copy link
Contributor

@softvision-miralobontiu I've spoken with @kglazko and she said it's okay to remove long_press_tapped.
As this issue's PR is already merged I've created a new issue for the removal. (#17481).

Besides the removal of long_press_tapped is there anything else that needs to be done on this issue?

@codrut-topliceanu codrut-topliceanu added this to Ready for Engineering (min-5 ; max-22) in Android Engineering Team Kanban board via automation Jan 15, 2021
@codrut-topliceanu codrut-topliceanu moved this from Ready for Engineering (min-5 ; max-22) to In Testing/QA in Android Engineering Team Kanban board Jan 15, 2021
@lobontiumira
Copy link

@codrut-topliceanu thanks!
Nope, the only reason the ticket was left opened was for the "contextual menu: long_press_tapped" event.
I'll close it.

Android Engineering Team Kanban board automation moved this from In Testing/QA to Done Jan 15, 2021
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Aug 4, 2021
…ozilla-mobile#16968)

* For mozilla-mobile#11580 - Tracks text selection context menu usage

Tracks Copy, Search, Select All and Share items from the text selection context menu. Uses AC's DefaultSelectionActionDelegate to achieve this.

Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature:ContextMenu Menu that appears when long-pressing on website content Feature:Telemetry Feature:TextSelection P2 Upcoming release
Projects
No open projects
Development

No branches or pull requests

7 participants