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

Commit

Permalink
Issue #11112 - Update empty collections string + tests. (#11538)
Browse files Browse the repository at this point in the history
  • Loading branch information
liuche authored Jun 12, 2020
1 parent a36ec72 commit 35fa445
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -514,13 +514,13 @@ private fun assertCollectionsHeader() =
.check(matches(withEffectiveVisibility(Visibility.VISIBLE)))

private fun assertNoCollectionsHeader() =
onView(allOf(withText("No collections")))
onView(allOf(withText("Collect the things that matter to you")))
.check(matches(withEffectiveVisibility(Visibility.VISIBLE)))

private fun assertNoCollectionsText() =
onView(
allOf(
withText("Collect the things that matter to you. To start, save open tabs to a new collection.")
withText("Group together similar searches, sites, and tabs for quick access later.")
)
)
.check(matches(withEffectiveVisibility(Visibility.VISIBLE)))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ import org.mozilla.fenix.home.OnboardingState
import org.mozilla.fenix.components.tips.Tip

val noCollectionMessage = AdapterItem.NoContentMessage(
R.string.no_collections_header,
R.string.collections_description
R.string.no_collections_header1,
R.string.no_collections_description1
)

// This method got a little complex with the addition of the tab tray feature flag
Expand Down
12 changes: 4 additions & 8 deletions app/src/main/res/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@
<string name="content_description_disable_private_browsing_button">Disable private browsing</string>
<!-- Placeholder text shown in the search bar before a user enters text -->
<string name="search_hint">Search or enter address</string>
<!-- No Open Tabs Message Header -->
<string name="no_open_tabs_header_2">No open tabs</string>
<!-- No Open Tabs Message Description -->
<string name="no_open_tabs_description">Your open tabs will be shown here.</string>
<!-- No Private Tabs Message Description -->
Expand Down Expand Up @@ -477,7 +475,7 @@
<!-- Open tabs menu item to share all tabs -->
<string name="tabs_menu_share_tabs">Share tabs</string>
<!-- Open tabs menu item to save tabs to collection -->
<string name="tabs_menu_save_to_collection">Save to collection</string>
<string name="tabs_menu_save_to_collection1">Save tabs to collection</string>
<!-- Content description (not visible, for screen readers etc.): Opens the tab menu when pressed -->
<string name="tab_menu">Tab menu</string>
<!-- Tab menu item to share the tab -->
Expand Down Expand Up @@ -686,16 +684,14 @@
<string name="delete_browsing_data_quit_off">Off</string>

<!-- Collections -->
<!-- Label to describe what collections are to a new user without any collections -->
<string name="collections_description">Collect the things that matter to you. To start, save open tabs to a new collection.</string>
<!-- Collections header on home fragment -->
<string name="collections_header">Collections</string>
<!-- Content description (not visible, for screen readers etc.): Opens the collection menu when pressed -->
<string name="collection_menu_button_content_description">Collection menu</string>
<!-- No Open Tabs Message Header -->
<string name="no_collections_header">No collections</string>
<!-- No Open Tabs Message Description -->
<string name="no_collections_description">Your collections will be shown here.</string>
<string name="no_collections_header1">Collect the things that matter to you</string>
<!-- Label to describe what collections are to a new user without any collections -->
<string name="no_collections_description1">Group together similar searches, sites, and tabs for quick access later.</string>
<!-- Title for the "select tabs" step of the collection creator -->
<string name="create_collection_select_tabs">Select Tabs</string>
<!-- Title for the "select collection" step of the collection creator -->
Expand Down

0 comments on commit 35fa445

Please sign in to comment.