blixt-wallet
blixt-wallet copied to clipboard
Add Future.Travel to Blixt
Add Future.Travel to Blixt
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
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?
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.
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.
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.