fix(invite) fix mailto links not working on Brave for iOS

This commit is contained in:
Avram Tudor 2021-04-15 13:52:11 +03:00 committed by GitHub
parent 0aef918c55
commit b74c8b5d1f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 22 deletions

View File

@ -97,18 +97,18 @@
border-top: none; border-top: none;
border-radius: 0 0 3px 3px; border-radius: 0 0 3px 3px;
& > * { .copy-invite-icon, .provider-icon {
display: flex;
justify-content: center;
align-items: center; align-items: center;
height: 40px;
width: 40px;
border-radius: 4px;
cursor: pointer; cursor: pointer;
display: flex;
height: 40px;
place-content: center;
width: 40px;
} }
&:hover > div:hover { &:hover > div:hover {
background-color: rgba(255, 255, 255, 0.2); background-color: rgba(255, 255, 255, 0.2);
border-radius: 4px;
} }
& > :not(:last-child) { & > :not(:last-child) {

View File

@ -13,7 +13,7 @@ import {
IconYahoo IconYahoo
} from '../../../../base/icons'; } from '../../../../base/icons';
import { Tooltip } from '../../../../base/tooltip'; import { Tooltip } from '../../../../base/tooltip';
import { copyText, openURLInBrowser } from '../../../../base/util'; import { copyText } from '../../../../base/util';
type Props = { type Props = {
@ -52,18 +52,6 @@ function InviteByEmailSection({ inviteSubject, inviteText, t }: Props) {
copyText(inviteText); copyText(inviteText);
} }
/**
* Opens an email provider containing the conference invite.
*
* @param {string} url - The url to be opened.
* @returns {Function}
*/
function _onSelectProvider(url) {
return function() {
openURLInBrowser(url, true);
};
}
/** /**
* Toggles the email invite drawer. * Toggles the email invite drawer.
* *
@ -112,10 +100,13 @@ function InviteByEmailSection({ inviteSubject, inviteText, t }: Props) {
content = { t(tooltipKey) } content = { t(tooltipKey) }
key = { idx } key = { idx }
position = 'top'> position = 'top'>
<div <a
onClick = { _onSelectProvider(url) }> className = 'provider-icon'
href = { url }
rel = 'noopener noreferrer'
target = '_blank'>
<Icon src = { icon } /> <Icon src = { icon } />
</div> </a>
</Tooltip> </Tooltip>
)) ))
} }