-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closes #8176 - Adds new string for add to top sites. #8209
Conversation
@@ -352,7 +352,7 @@ sealed class Event { | |||
enum class Item { | |||
SETTINGS, LIBRARY, HELP, DESKTOP_VIEW_ON, DESKTOP_VIEW_OFF, FIND_IN_PAGE, NEW_TAB, | |||
NEW_PRIVATE_TAB, SHARE, REPORT_SITE_ISSUE, BACK, FORWARD, RELOAD, STOP, OPEN_IN_FENIX, | |||
SAVE_TO_COLLECTION, ADD_TO_FIREFOX_HOME, ADD_TO_HOMESCREEN, QUIT, READER_MODE_ON, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we update this as well? https://github.com/mozilla-mobile/fenix/blob/master/app/metrics.yaml#L81
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
57463d8
to
6dcff4e
Compare
Codecov Report
@@ Coverage Diff @@
## master #8209 +/- ##
============================================
+ Coverage 18.67% 18.67% +<.01%
Complexity 452 452
============================================
Files 316 316
Lines 12331 12329 -2
Branches 1619 1619
============================================
Hits 2303 2303
+ Misses 9829 9827 -2
Partials 199 199
Continue to review full report at Codecov.
|
@@ -221,15 +221,12 @@ class DefaultToolbarMenu( | |||
onItemTapped.invoke(ToolbarMenu.Item.RequestDesktop(checked)) | |||
} | |||
|
|||
private val addToFirefoxHome = BrowserMenuImageText( | |||
label = context.getString( | |||
R.string.browser_menu_add_to_firefox_home, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can remove "browser_menu_add_to_firefox_home" string as well.
Pull Request checklist
After merge
To download an APK when reviewing a PR: