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

Add translations for open tabs in search autosuggest #5065

Conversation

anikiki
Copy link
Contributor

@anikiki anikiki commented Sep 25, 2024

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

Description

Translations.

Steps to test this PR

  • Install from this branch.
  • Open a few tabs. E.g www.gardenersworld.com
  • Search for something that will match an open tab. E.g garden
  • Notice the "Switch to Tab · " text before the URL in the suggestion.

UI changes (Added Switch to Tab string)

Before After
switch_to_tab_before switch_to_tab_after

@anikiki anikiki changed the title Added translations. Add translations for open tabs in search autosuggest Sep 25, 2024
@anikiki anikiki marked this pull request as ready for review September 25, 2024 16:59
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 1ecf111 to 328e2b4 Compare September 26, 2024 11:23
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from 3f6cf64 to 17de826 Compare September 26, 2024 11:23
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 328e2b4 to 2491fda Compare September 26, 2024 11:45
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from b1f4598 to de8298f Compare September 26, 2024 11:45
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 2491fda to e18e755 Compare September 26, 2024 12:10
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from de8298f to 29e54bb Compare September 26, 2024 12:10
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from e18e755 to 6fbc6f3 Compare September 26, 2024 12:12
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from 29e54bb to 0994f2e Compare September 26, 2024 12:12
Copy link
Contributor

@CrisBarreiro CrisBarreiro 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!

@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 6fbc6f3 to 880ec9e Compare October 10, 2024 15:29
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from 1ee6d24 to 12256db Compare October 10, 2024 15:29
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 880ec9e to c2fd1de Compare October 10, 2024 15:39
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from c46cf24 to 5d0ed05 Compare October 10, 2024 15:39
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from c2fd1de to 3abd869 Compare October 10, 2024 15:45
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from 5d0ed05 to 871743c Compare October 10, 2024 15:45
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from 3abd869 to e159b67 Compare October 11, 2024 16:07
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from e62d927 to 26275f2 Compare October 11, 2024 16:07
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from e159b67 to bc72676 Compare October 18, 2024 12:10
Translate strings to values-ru
Translate strings to values-da
Translate strings to values-cs
Translate strings to values-fr
Translate strings to values-lt
Translate strings to values-ro
Translate strings to values-sk
Translate strings to values-ru
Translate strings to values-et
Translate strings to values-fi
Translate strings to values-sl
Translate strings to values-nl
Translate strings to values-hr
Translate strings to values-hu
Translate strings to values-sv
Translate strings to values-da
Translate strings to values-fr
Translate strings to values-lt
Translate strings to values-pt
Translate strings to values-cs
Translate strings-autofill-impl to values-nl
Translate strings-autofill-impl to values-it
Translate strings-autofill-impl to values-ro
Translate strings-autofill-impl to values-sk
Translate strings-autofill-impl to values-sv
Translate strings-autofill-impl to values-bg
Translate strings-autofill-impl to values-fr
Translate strings-autofill-impl to values-sl
Translate strings-autofill-impl to values-lt
Translate strings-autofill-impl to values-el
Translate strings-autofill-impl to values-pt
Translate strings-autofill-impl to values-et
Translate strings-autofill-impl to values-nb
Translate strings-autofill-impl to values-de
Translate strings-autofill-impl to values-ru
Translate strings-autofill-impl to values-tr
Translate strings-autofill-impl to values-hu
Translate strings-autofill-impl to values-fi
Translate strings-autofill-impl to values-lv
Translate strings-autofill-impl to values-pl
Translate strings-autofill-impl to values-da
Translate strings-autofill-impl to values-es
Translate strings-autofill-impl to values-cs
Translate strings-autofill-impl to values-nl
Translate strings-autofill-impl to values-it
Translate strings-autofill-impl to values-ro
Translate strings-autofill-impl to values-sk
Translate strings-autofill-impl to values-sv
Translate strings-autofill-impl to values-bg
Translate strings-autofill-impl to values-fr
Translate strings-autofill-impl to values-sl
Translate strings-autofill-impl to values-lt
Translate strings-autofill-impl to values-el
Translate strings-autofill-impl to values-pt
Translate strings-autofill-impl to values-et
Translate strings-autofill-impl to values-nb
Translate strings-autofill-impl to values-de
Translate strings-autofill-impl to values-ru
Translate strings-autofill-impl to values-tr
Translate strings-autofill-impl to values-hu
Translate strings-autofill-impl to values-fi
Translate strings-autofill-impl to values-lv
Translate strings-autofill-impl to values-pl
Translate strings-autofill-impl to values-da
Translate strings-autofill-impl to values-es
Translate strings-autofill-impl to values-cs
Translate strings-duckplayer to values-fi
Translate strings-duckplayer to values-pl
Translate strings-duckplayer to values-sl
Translate strings-duckplayer to values-da
Translate strings-duckplayer to values-de
Translate strings-duckplayer to values-it
Translate strings-duckplayer to values-es
Translate strings-duckplayer to values-lt
Translate strings-duckplayer to values-et
Translate strings-duckplayer to values-sv
Translate strings-duckplayer to values-tr
Translate strings-duckplayer to values-pt
Translate strings-duckplayer to values-lv
Translate strings-duckplayer to values-hu
Translate strings-duckplayer to values-fr
Translate strings-duckplayer to values-hr
Translate strings-duckplayer to values-ru
Translate strings-duckplayer to values-nb
Translate strings-duckplayer to values-sk
Translate strings-duckplayer to values-bg
Translate strings-duckplayer to values-ro
Translate strings-duckplayer to values-cs
Translate strings-duckplayer to values-nl
@anikiki anikiki force-pushed the feature/ana/add_pixels_for_open_tabs_in_search_autosuggest branch from bc72676 to 82707b3 Compare October 21, 2024 15:15
@anikiki anikiki force-pushed the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch from e65d5a4 to f7a8db6 Compare October 21, 2024 15:15
Task/Issue URL:
https://app.asana.com/0/72649045549333/1208287265878043/f

### Description
Added feature flag for switch to tab. Removed unneeded quick insert
icons. Formatted urls in autosuggest. Added dividers between autosuggest
sections.
Updated color on switch to tab icon.

### Steps to test this PR

_Feature 1_
- [x] Install from this branch.
- [x] Open a few tabs.
- [x] Search for something that matches the open tabs. Notice there's a
divides between the top hits and search items. Notice there's a divider
between the search items and the bottom section.
- [x] Notice the quick insert icon is available only for search
suggestions.
- [x] Notice all URLs in suggestions don't have `http://www.`,
`htps://www.`, `www.` as prefix and they don't have a trailing `/`.
- [x] Notice that if you turn off the `autocompleteTabs` from `Feature
Flag Inventory` you don't see the Switch to tab suggestions.

### UI changes
| Divider dark node  | Divider light mode |
| ------ | ----- |

|![divider_light_mode](https://github.com/user-attachments/assets/30802a7b-6936-46b2-9b79-56d3af1efc92)|![divider_dark_mode](https://github.com/user-attachments/assets/c2f277fc-7e9b-41f8-b6f9-938916c68034)|

|No top divider | Bottom divider |
| ------ | ----- |

|![no_top_divider](https://github.com/user-attachments/assets/55467885-89d4-4e7d-82c4-2dbc8e41ecd6)|![bottom_divider](https://github.com/user-attachments/assets/1be711bf-c731-4379-90a9-f53c2995f914)|
@anikiki anikiki merged commit d2c56c7 into feature/ana/add_pixels_for_open_tabs_in_search_autosuggest Oct 22, 2024
4 checks passed
@anikiki anikiki deleted the feature/ana/add_translations_for_open_tabs_in_search_autosuggest_l10n branch October 22, 2024 11:42
anikiki added a commit that referenced this pull request Oct 25, 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]>
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.

3 participants