Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update QrEyeStyle Rounded, QrDataModule Rounded & Logo Background #193

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

muhammadsaddamnur
Copy link

@muhammadsaddamnur muhammadsaddamnur commented Nov 3, 2022

I was trying to QrEyeStyle Rounded, QrDataModuleShape Rounded & Logo Background
like this :
Screenshot_1667466328

Please review! Thank you

@muhammadsaddamnur muhammadsaddamnur changed the title Update Rounded QrEyeStyle, QrDataModuleShape rounded & Logo background Update QrEyeStyle Rounded, QrDataModule Rounded & Logo Background Nov 3, 2022
@alexanderkind
Copy link
Contributor

Wonderful! Waiting for update)

@justkawal
Copy link

justkawal commented Dec 13, 2022

@muhammadsaddamnur Awesome PR btw, Waiting for it to get merged.
It would be awesome if you could provide some test-cases for it with golden generated images....

@NAME-NikhilPatil
Copy link

Awesome😍

@mahdiqdi
Copy link

after getting this PR in my pubspec it says QrImage is not a widget.plz help me to have this feature in my project.plz merge this pr

@Amphaal
Copy link

Amphaal commented Feb 6, 2023

Many thanks @muhammadsaddamnur for this PR ! I wish this repo was still maintained tho...

@muhammadsaddamnur
Copy link
Author

muhammadsaddamnur commented Jul 6, 2023

I have added golden tests and made several updates to the example in this pull request. Please review.
@justkawal

@muhammadsaddamnur Awesome PR btw, Waiting for it to get merged. It would be awesome if you could provide some test-cases for it with golden generated images....

@esentis
Copy link

esentis commented Jul 11, 2023

Any chance of this being merged ?

@vitoramaral10
Copy link

I need this, when will it be merged?

@husainazkas
Copy link

still waiting :)

@onlyamessenger
Copy link

Also waiting, this would be awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants