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

Search term lost when tapping back into search field #1205

Closed
Verdi opened this issue Mar 26, 2019 · 3 comments
Closed

Search term lost when tapping back into search field #1205

Verdi opened this issue Mar 26, 2019 · 3 comments
Assignees
Labels
eng:qa:verified QA Verified Feature:Search needs:ac Needs Android Component Work

Comments

@Verdi
Copy link

Verdi commented Mar 26, 2019

Steps to reproduce

  1. Search for anything (e.g. Firefox) and navigate to the search engine results page.
  2. When on the search results page, tap the Fenix search field.

Expected behavior

My search query, "Firefox" should be in the search field, and I should see search and history suggestions related to it.

Actual behavior

My search query is gone and the url of the search engine results page is in the search field and it's the top suggestion.

Device information

  • Android device: Nokia 3.1
  • Fenix version: 1.0.1913

Priority

I think this is a blocker.

┆Issue is synchronized with this Jira Task

@lime124 lime124 added the needs:UX-feedback Needs UX Feedback label Mar 28, 2019
@lime124 lime124 mentioned this issue Mar 28, 2019
@vesta0 vesta0 added Feature:Search P1 Current sprint and removed Feature:Search labels Apr 1, 2019
@vesta0 vesta0 added this to the MVP Backlog milestone Apr 1, 2019
@vesta0 vesta0 added needs:ac Needs Android Component Work needs:product and removed needs:UX-feedback Needs UX Feedback labels Apr 2, 2019
@ghost
Copy link

ghost commented Apr 10, 2019

This functionality already exists in A-C, (you can see it in action in Reference Browser), but it means that Fenix needs to start using the feature-toolbar to get this functionality. It is unclear what the reason was for Fenix not using this component.

Did @boek work on this earlier in the year? Can we refactor the current code to start using the component?

If not then Fenix will have to implement this functionality directly using the available (navigation) callback - but I think we prefer to use the component since that will be more consistent between our applications then.

@sblatz
Copy link
Contributor

sblatz commented Apr 17, 2019

After quite a bit of work on this, I was able to narrow it down to being an issue with the way a-c is persisting search terms in their session. While I have a "hacky" implementation working for Fenix, I spoke with @pocmo, and he's going to fix this internally in a-c so that everyone benefits from the fix.

This ticket will now be blocked on mozilla-mobile/android-components#2771

@sblatz sblatz added 🙅 waiting Issues that are blocked or has dependencies that are not ready and removed in progress labels Apr 17, 2019
@project-bot project-bot bot moved this from In Progress to Blocked in Fenix Sprint Kanban Apr 17, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Apr 23, 2019
@vesta0 vesta0 removed the 🙅 waiting Issues that are blocked or has dependencies that are not ready label Apr 24, 2019
@vesta0 vesta0 moved this from Blocked to In Progress in Fenix Sprint Kanban Apr 24, 2019
@sblatz sblatz added the eng:qa:needed QA Needed label Apr 24, 2019
@project-bot project-bot bot moved this from In Progress to Ready for QA in Fenix Sprint Kanban Apr 24, 2019
@kbrosnan kbrosnan added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 26, 2019
@kbrosnan
Copy link
Contributor

This is working as expected. Tapping into the address bar after performing a search brings back the search terms. Navigating to a new page no longer displays the search terms. Swipe killing the browser clears the search terms. Tested on a Pixel 3 and Galaxy S6.

@ghost ghost removed the P1 Current sprint label Apr 26, 2019
@kbrosnan kbrosnan moved this from Ready for QA to Done in Fenix Sprint Kanban Apr 26, 2019
@kbrosnan kbrosnan removed the Search label Apr 27, 2019
@bifleming bifleming removed this from Done in Fenix Sprint Kanban May 7, 2019
@data-sync-user data-sync-user changed the title Search term lost when tapping back into search field FNX2-16779 ⁃ Search term lost when tapping back into search field Aug 2, 2020
@data-sync-user data-sync-user changed the title FNX2-16779 ⁃ Search term lost when tapping back into search field FNX3-15089 ⁃ Search term lost when tapping back into search field Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-15089 ⁃ Search term lost when tapping back into search field FNX-4871 ⁃ Search term lost when tapping back into search field Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-4871 ⁃ Search term lost when tapping back into search field FNX2-16779 ⁃ Search term lost when tapping back into search field Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX2-16779 ⁃ Search term lost when tapping back into search field Search term lost when tapping back into search field May 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:qa:verified QA Verified Feature:Search needs:ac Needs Android Component Work
Projects
None yet
Development

No branches or pull requests

6 participants