lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

[Feature] Add logo to the QR codes for invoices

Open volfiros opened this issue 2 years ago • 12 comments

Feature description

Adding the Alby logo to the QR code used for invoices

Describe the solution

Currently we are using the react-qr-code package for creating QR code with invoice instead we can use the react-qrcode-logo package to add a logo to the QR codes.

Describe alternatives

No response

Additional context

image

Are you working on this?

Yes

volfiros avatar Sep 18 '23 05:09 volfiros

@stackingsaunter @rolznz @bumi any thoughts on this?

volfiros avatar Sep 18 '23 05:09 volfiros

@Rithvik-padma I am not sure this is necessary now. We just merged a PR to add the user's account avatar to the QR code. Can you please checkout master and try it?

rolznz avatar Sep 18 '23 09:09 rolznz

@Rithvik-padma I am not sure this is necessary now. We just merged a PR to add the user's account avatar to the QR code. Can you please checkout master and try it?

Ohh I was using a LND test account so I couldn't find that, but we can add the alby logo for non-alby users instead of the avatar (better than a plain QR code), only if it sounds necessary.

volfiros avatar Sep 18 '23 10:09 volfiros

I think this can be nice-have if it won't affect scanning. I would use Alby head with white stroke here though so it's visable, or in a white circle. It would be great If you could match the design style here

CleanShot 2023-09-19 at 03 22 11@2x

Head

stackingsaunter avatar Sep 19 '23 01:09 stackingsaunter

Yeah, sure I will see that the design style matches.

volfiros avatar Sep 19 '23 02:09 volfiros

@rolznz Can you assing me please ?

AnamuddinAhmad avatar Oct 02 '23 05:10 AnamuddinAhmad

@Rithvik-padma did you work on this one?

rolznz avatar Oct 02 '23 07:10 rolznz

Yes, I am working on it.

On Mon, 2 Oct, 2023, 12:48 PM Roland, @.***> wrote:

@Rithvik-padma https://github.com/Rithvik-padma did you work on this one?

— Reply to this email directly, view it on GitHub https://github.com/getAlby/lightning-browser-extension/issues/2757#issuecomment-1742511168, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4MOVRRL7ET5HYP76B4LQ2LX5JTC3ANCNFSM6AAAAAA44EWDLI . You are receiving this because you commented.Message ID: @.***>

AnamuddinAhmad avatar Oct 02 '23 10:10 AnamuddinAhmad

@Rithvik-padma did you work on this one?

Yes I have made the changes locally, will make a PR soon. Thanks!

volfiros avatar Oct 02 '23 12:10 volfiros

@stackingsaunter I feel the image you provided is too small considering the QR code size

RLpIMRg2iS this is icon you have provided but this is small and of size 32x32 ig.

Tk60gUEYmV this is of size 64x64 (expanded version)

should I use the icon you have provided or if not can you provide the icon I should use here. Thanks!

volfiros avatar Oct 02 '23 13:10 volfiros

@Rithvik-padma can you use an svg version of the icon?

rolznz avatar Oct 03 '23 03:10 rolznz

@Rithvik-padma can you use an svg version of the icon?

Yes, I am using a svg version only, okay so anyway it will not affect the quality. Thanks!

volfiros avatar Oct 03 '23 03:10 volfiros