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

Tab swiping: Pixels #5400

Merged
merged 105 commits into from
Dec 20, 2024
Merged

Conversation

0nko
Copy link
Member

@0nko 0nko commented Dec 17, 2024

Task/Issue URL: https://app.asana.com/0/72649045549333/1208648123066972/f

Description

This PR adds pixel for tab swiping.

Steps to test this PR

Change tabs with swiping

  • Add a few tabs
  • Swipe tabs 3 times
  • Verify m_swipe_tabs_used pixel is fired 3 times
  • Verify m_swipe_tabs_used_daily pixel is fired once

Change tabs with not swiping

  • Add a few tabs
  • Long press on a tabs button to open a new tab
  • Go to tab switch
  • Select a different tab
  • Add a new tab from the menu
  • Verify m_swipe_tabs_used pixel is never fired

0nko and others added 30 commits November 11, 2024 10:43
@0nko 0nko changed the title Feature/ondrej/swiping tabs pixels Tab swiping: Pixels Dec 17, 2024
@anikiki anikiki self-assigned this Dec 18, 2024
Copy link
Contributor

@anikiki anikiki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and works as expected! 🎉

Base automatically changed from feature/ondrej/swiping-tabs-ui to feature/ondrej/swiping-tabs December 20, 2024 11:36
0nko added 2 commits December 20, 2024 12:47
…g-tabs-pixels

# Conflicts:
#	app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt
#	app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt
#	app/src/main/java/com/duckduckgo/app/browser/tabs/DefaultTabManager.kt
#	app/src/main/java/com/duckduckgo/app/browser/tabs/adapter/TabPagerAdapter.kt
#	app/src/test/java/com/duckduckgo/app/browser/BrowserViewModelTest.kt
@0nko 0nko merged commit abf45e8 into feature/ondrej/swiping-tabs Dec 20, 2024
4 checks passed
@0nko 0nko deleted the feature/ondrej/swiping-tabs-pixels branch December 20, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants