Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-09-20 11:34:34 +08:00
Code Issues Projects Releases Packages Wiki Activity

Remove the “Twitter image” option in Prefs > Signature, it does not work with X

Browse source
This commit is contained in:
Ben Gotow 2023-11-14 15:11:01 -06:00
parent a1998fcc94
commit 3d1a513c4b
2 changed files with 2 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
app/internal_packages/composer-signature/lib/constants.ts
View file

@ -49,7 +49,7 @@ export const DataShape = [
},
{
key: 'instagramURL',
label: localized('Instagram URL')
label: localized('Instagram URL'),
},
{
key: 'tintColor',
@ -95,12 +95,6 @@ export const ResolveSignatureData = data => {
data.photoURL = `https://www.gravatar.com/avatar/${hash}/?s=160&msw=160&msh=160`;
}
if (data.photoURL === 'twitter') {
data.photoURL = `https://twitter.com/${
data.twitterHandle
}/profile_image?size=original&msw=128&msh=128`;
}
if (data.photoURL === 'company') {
const domain =
(data.websiteURL && URL.parse(data.websiteURL).hostname) ||
@ -118,7 +112,6 @@ export const ResolveSignatureData = data => {
}
}
return data;
};

3
app/internal_packages/composer-signature/lib/signature-photo-picker.tsx
View file

@ -150,7 +150,7 @@ export default class SignaturePhotoPicker extends React.Component<
const { isDropping, isUploading } = this.state;
let source = data.photoURL || '';
if (!['gravatar', 'twitter', 'company', ''].includes(source)) {
if (!['gravatar', 'company', ''].includes(source)) {
source = 'custom';
}
@ -203,7 +203,6 @@ export default class SignaturePhotoPicker extends React.Component<
>
<option value="">{localized('None')}</option>
<option value="gravatar">{localized('Gravatar Profile Photo')}</option>
<option value="twitter">{localized('Twitter Profile Image')}</option>
<option value="company">{localized('Company / Domain Logo')}</option>
<option disabled>──────────</option>
<option value="custom">{localized('Custom Image…')}</option>