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

make popup appear faster #297

Merged
merged 2 commits into from
Feb 17, 2023
Merged

make popup appear faster #297

merged 2 commits into from
Feb 17, 2023

Conversation

capaj
Copy link
Collaborator

@capaj capaj commented Feb 17, 2023

  • unify getDomainNameAndTldFromUrl in shared
    this fixes a scenario when we would be autocompleting, but popup was showing 0 logins because popup was filtering by whole hostname, not just domain+TLD

  • make popup appear faster

@capaj capaj added the enhancement New feature or request label Feb 17, 2023
@capaj capaj requested a review from sleaper February 17, 2023 01:04
@capaj capaj changed the title unify getDomainNameAndTldFromUrl in shared make popup appear faster Feb 17, 2023
Copy link
Collaborator

@sleaper sleaper left a comment

Choose a reason for hiding this comment

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

🚀

@capaj capaj merged commit 6f5f920 into main Feb 17, 2023
capaj added a commit that referenced this pull request May 11, 2023
* unify getDomainNameAndTldFromUrl in shared

* make popup init faster

Former-commit-id: 6f5f920
capaj added a commit that referenced this pull request May 11, 2023
* unify getDomainNameAndTldFromUrl in shared

* make popup init faster

Former-commit-id: 2808cfdccfd51a3f1c170ddaa805092c809b1555 [formerly 6f5f920]
Former-commit-id: 6c5619d
capaj added a commit that referenced this pull request May 11, 2023
* unify getDomainNameAndTldFromUrl in shared

* make popup init faster

Former-commit-id: 2808cfdccfd51a3f1c170ddaa805092c809b1555 [formerly 2808cfdccfd51a3f1c170ddaa805092c809b1555 [formerly 6f5f920]]
Former-commit-id: 6451315c5d227d8f309f7f223755e8b3fdf2965e [formerly 6c5619d]
Former-commit-id: 833e88f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants