blixt-wallet icon indicating copy to clipboard operation
blixt-wallet copied to clipboard

Add Future.Travel to Blixt

Open FutureTravelATM opened this issue 3 years ago • 5 comments

Add Future.Travel to Blixt

FutureTravelATM avatar Nov 01 '21 05:11 FutureTravelATM

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hsjoberg/blixt-wallet/o6HinwEAUAZouJ13YJxn42HYFKqD
✅ Preview: https://blixt-wallet-git-fork-futuretravelatm-master-hsjoberg.vercel.app

vercel[bot] avatar Nov 01 '21 05:11 vercel[bot]

Thank you for your pull request @FutureTravelATM. I think the code looks great!

I have a concern regarding image itself, as can be observed via the Vercel preview, the text in the image is too small to be readable. Would it be possible just use FutureTravel's logotype instead and skip the text?

hsjoberg avatar Nov 05 '21 00:11 hsjoberg

Let us put a halo circle around it. Our name is our logo.

Thx.

Sent from my iPhone

On 5 Nov 2021, at 7:07 am, Hampus Sjöberg @.***> wrote:

 Thank you for your pull request @FutureTravelATM. I think the code looks great!

I have a concern regarding image itself, as can be observed via the Vercel preview, the text in the image is too small to be readable. Would it be possible just use FutureTravel's logotype instead and skip the text?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

FutureTravelATM avatar Nov 05 '21 00:11 FutureTravelATM

Here are 2 options, as our name is our Logo.

Plse use the one with the white background.

Let's see if that works.

Thanks,

David

On Fri, Nov 5, 2021 at 7:07 AM Hampus Sjöberg @.***> wrote:

Thank you for your pull request @FutureTravelATM https://github.com/FutureTravelATM. I think the code looks great!

I have a concern regarding image itself, as can be observed via the Vercel preview https://blixt-wallet-lljkil96h-hsjoberg.vercel.app, the text in the image is too small to be readable. Would it be possible just use FutureTravel's logotype instead and skip the text?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/hsjoberg/blixt-wallet/pull/391#issuecomment-961526179, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSLAD7WNKRW4INAYQWTTNDUKMN3FANCNFSM5HDGTOEQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- David Watson General Director Industry Travel Asia/Future.Travel +84 28 3999 0706 Ext 101 +84 903 820 285

@.*** IATA 37-3 0237 0 Future.Travel

*Industry Travel Asia *

/ Future.TravelWe specialise in promoting industry education and growth through travel experiences with our partners. Future.Travel - online portal to worldwide travel options. Indochina PSA for Costa Cruises Lines.

FutureTravelATM avatar Nov 05 '21 08:11 FutureTravelATM

Hi @FutureTravelATM, I don't think the logo looks visually pleasing because of the space constraints we're dealing with here in the transaction log. I still suggest to cut out the text considering the name will be shown just to right of the image.

image

hsjoberg avatar Apr 20 '22 14:04 hsjoberg