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

Receiving tab from desktop app should show correct legend on notification banner #37058

Open
Uni-verse opened this issue Sep 13, 2022 · 2 comments
Assignees
Labels
enhancement OS/iOS Fixes related to iOS browser functionality QA/Yes

Comments

@Uni-verse
Copy link
Collaborator

Description:

Receiving tab from desktop app should show correct legend on notification banner

Steps to Reproduce

  1. Connect desktop and mobile apps to sync chain
  2. Send page to mobile app from desktop
  3. Observe notification banner legend on mobile

Actual result:

Example Example Example
IMG_0056 IMG_0564 Screen Shot 2022-09-13 at 11 26 41 AM

Expected result:
When sending from desktop app, it should show desktop legend on mobile client.

Screen Shot 2022-09-13 at 11 25 15 AM

Brave Version:
1.43.x

Device details:
iPhone, iPad

@soner-yuksel
Copy link
Member

So I tried to handle this case but it turns out changing the icon according to device will not be possible.

However,

I tried to match sync cache ids for receiver and sender but they didn't match with the ones from device list for sender list and receiver device. So the only option is to use what Chromium provides under send_tab_to_self::SendTabToSelfEntry

From Chromium we receive send_tab_to_self::SendTabToSelfEntry when we receive a tab and this information contains

URL - Title - Time when it is shared - Device Name

When desktop receives a Tab from another they utilize all the information and use a popup like this

Screen Shot 2022-09-23 at 12 56 23 PM

They use title url and device name to to utilize the information.

So what I believe even we cant change the icon we should improve the Toast which is showing "Tab Received" using additional information like URL - Title - Device Name.

@stoletheminerals also mentioned this in original PR brave/brave-ios#5809 (comment)

Please tell me what do you think about it ?
@iccub @stoletheminerals @Uni-verse

@stoletheminerals
Copy link

So what I believe even we cant change the icon we should improve the Toast which is showing "Tab Received" using additional information like URL - Title - Device Name.

I agree. Ideally, we should have parity with Desktop and Android. Showing URL should be a bare minimum.

@soner-yuksel soner-yuksel transferred this issue from brave/brave-ios Mar 25, 2024
@kylehickinson kylehickinson added the OS/iOS Fixes related to iOS browser functionality label Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement OS/iOS Fixes related to iOS browser functionality QA/Yes
Projects
None yet
Development

No branches or pull requests

4 participants