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

[Implementation] Android: Include existing open tabs in search autosuggest #5018

Conversation

@anikiki anikiki changed the title Initial commit. [Implementation] Android: Include existing open tabs in search autosuggest Sep 16, 2024
@anikiki anikiki force-pushed the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch from fc8300b to 08c8e0d Compare September 26, 2024 11:23
@anikiki anikiki force-pushed the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch 3 times, most recently from 64aaecd to c8cd3d6 Compare October 11, 2024 16:06
@anikiki anikiki force-pushed the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch 2 times, most recently from ba10a54 to 9e0586d Compare October 21, 2024 15:15
@anikiki anikiki marked this pull request as ready for review October 22, 2024 11:44
@anikiki anikiki force-pushed the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch 4 times, most recently from b6243b2 to a2db1d7 Compare October 25, 2024 11:34
Copy link
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

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

LGTM!

anikiki and others added 3 commits October 25, 2024 15:56
Task/Issue URL:

Added all open tabs to suggestions.

All opened tabs (no checks on the query) are placed above history, for
simplicity.
Matching and ranking to be done separately.

- [x] Install from this branch.
- [x] Open a few websites in different tabs.
- [x] Search for something and notice the "Tab" suggestions. There is no
matching done, only check the UI (the switch to tab text is not yet
present).

| Example 1 (with history)  | Example 2 (without history) |
| ------ | ----- |

|![Screenshot_20240916_215653](https://github.com/user-attachments/assets/af875653-1dd1-4376-bda7-9f753c4a354c)|![Screenshot_20240916_220717](https://github.com/user-attachments/assets/27a7a05e-c0d1-43d2-ae68-37db43b71f20)|

---
- To see the specific tasks where the Asana app for GitHub is being
used, see below:
  - https://app.asana.com/0/0/1208264037618740

---------

Co-authored-by: Dax The Translator <[email protected]>
Updated alignment.
@anikiki anikiki force-pushed the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch from a2db1d7 to 030b78b Compare October 25, 2024 14:57
@anikiki anikiki merged commit 6415f0f into develop Oct 25, 2024
6 checks passed
@anikiki anikiki deleted the feature/ana/implementation_android_include_existing_open_tabs_in_search_autosuggest branch October 25, 2024 15:12
aitorvs pushed a commit that referenced this pull request Nov 6, 2024
…ggest (#5018)

Task/Issue URL:
https://app.asana.com/0/1200581511062568/1208260875798966/f

### Description
The full implementation will span across a few tasks.

This is the main task for the implementation of [Android: Include
existing open tabs in search
autosuggest](https://app.asana.com/0/715106103902962/1208040193300233/f)

### Steps to test this PR

See:
- #5019
- #5042
- #5063
- #5064
- #5065
- #5105
- #5123

See scenarios and screenshots in
https://app.asana.com/0/1200581511062568/1208264037618741/f

---------

Co-authored-by: Dax The Translator <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants