Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

mailto: notifications display "null" rather than the domain when opening in new tabs #4198

Closed
Sh1d0w opened this issue Sep 22, 2016 · 9 comments · Fixed by #10668
Closed

Comments

@Sh1d0w
Copy link

Sh1d0w commented Sep 22, 2016

Test plan

#10668 (comment)


Did you search for similar issues before submitting this one?

No

Describe the issue you encountered:

Clicking mailto link produces the following message:

Allow ⁨null⁩ to ⁨open an external application⁩?

- Brave Version: 0.12.1 - Steps to reproduce: 1. Open brave.com 2. Go to footer section 3. Click Support link in the footer - Screenshot if needed: - Any related issues: #4003
@Sh1d0w Sh1d0w added the bug label Sep 22, 2016
@Sh1d0w Sh1d0w added this to the 0.12.3dev milestone Sep 22, 2016
@bbondy
Copy link
Member

bbondy commented Sep 26, 2016

@Sh1d0w could you re-verify this with master tip? I can't reproduce? If you can still reproduce please let me know the page and link that you can reproduce from. Thanks!

@bbondy bbondy modified the milestones: 0.12.4dev, 0.12.3dev Sep 26, 2016
@Sh1d0w
Copy link
Author

Sh1d0w commented Sep 27, 2016

@bbondy Hey Brian. Yes it happens on the latest master. I think it can be Linux issue only, currently don't have Mac or Win PC to confirm that, but it happens under Linux.

The steps to reproduce are the same as stated in the issue.

@bbondy bbondy removed this from the 0.12.4dev milestone Oct 4, 2016
@Sh1d0w
Copy link
Author

Sh1d0w commented Oct 9, 2016

@bbondy Since update to version 0.12.4 now this message appears even if I try to open a magnet torrent link:

Allow ⁨null⁩ to ⁨open an external application⁩?

@kjozwiak kjozwiak changed the title Bug with links of type mailto: mailto: notifications display "null" rather than the domain when opening in new tabs Aug 25, 2017
@kjozwiak
Copy link
Member

kjozwiak commented Aug 25, 2017

I ran into this issue while helping a community memeber. I noticed that rather than the correct domain appearing under the notification, "Null" was being displayed. This happens when a website uses target="_blank" for their mailto: links or the user opens the mailto: link in a new tab.

STR:

  • Open the latest version of Brave
  • Visit https://brave.com/ and scroll to the bottom
  • Under Contact Us, right click on an email and select "Open Link in New Tab"

screen shot 2017-08-25 at 12 16 55 pm

Reproduced with the latest build of Brave:

Brave 0.18.23 | rev 36ae2ec
Muon 4.3.10
libchromiumcontent 60.0.3112.90
V8 6.0.286.52

Reproducible on the following platforms:

  • macOS 10.12.6 x64 - Reproduced
  • Win 10 x64 Build Build 15063 - Reproduced

@diracdeltas, could you please take a look?

@kjozwiak kjozwiak added this to the 0.21.x (Nightly Channel) milestone Aug 25, 2017
diracdeltas added a commit that referenced this issue Aug 25, 2017
in the permission notification bar

fix #4198

Test Plan:
1. go to brave.com
2. scroll down to the contact section
4. right click on one of the email addresses to open in a new tab
5. the notification bar should say, 'Allow Brave Browser to open an external application?'
@bbondy bbondy modified the milestones: 0.21.x (Nightly Channel), 0.19.x (Beta Channel) Sep 20, 2017
bbondy pushed a commit that referenced this issue Sep 20, 2017
in the permission notification bar

fix #4198

Test Plan:
1. go to brave.com
2. scroll down to the contact section
4. right click on one of the email addresses to open in a new tab
5. the notification bar should say, 'Allow Brave Browser to open an external application?'
bbondy pushed a commit that referenced this issue Sep 20, 2017
in the permission notification bar

fix #4198

Test Plan:
1. go to brave.com
2. scroll down to the contact section
4. right click on one of the email addresses to open in a new tab
5. the notification bar should say, 'Allow Brave Browser to open an external application?'
@bbondy
Copy link
Member

bbondy commented Sep 20, 2017

Pulled this into 0.19.x to another merge easier.

0.19.x: 00ccec4
0.20.x: ef9374f
master: 3383ff6

@srirambv
Copy link
Collaborator

No notification on Windows
4198

@srirambv srirambv reopened this Sep 21, 2017
@LaurenWags
Copy link
Member

I see the same as @srirambv on MacOS.

@diracdeltas
Copy link
Member

The notification not appearing at all seems to be a separate issue. This change only changes the name of the domain in the notification.

@diracdeltas
Copy link
Member

^ opened #11075

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.