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

Bucket fill breaks with moved canvas #20

Open
7 tasks done
ManDay opened this issue Jan 7, 2025 · 2 comments
Open
7 tasks done

Bucket fill breaks with moved canvas #20

ManDay opened this issue Jan 7, 2025 · 2 comments
Labels
bug Something is not working

Comments

@ManDay
Copy link

ManDay commented Jan 7, 2025

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.0.0

Affected Android/Custom ROM version

DivestOS

Affected device model

Sargo/P3A

How did you install the app?

F-Droid / IzzyOnDroid

Steps to reproduce the bug

  1. Draw something
  2. Move and scale the canvas
  3. Use the bucket tool

Expected behavior

The bucket tool should fill the entire screen in a manner consistent with its definition.

Actual behavior

The bucket tool only acts when clicking in the area corresponding to the unmoved canvas and then fills that area with pixels corresponding to the actually expected image.

Screenshots/Screen recordings

No response

Additional information

  1. Thanks for publishing it, long awaited tool in the line 🙂
  2. There is no FAQ in "Menu → About"
@ManDay ManDay added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Jan 7, 2025
@naveensingh naveensingh removed the needs triage Issue is not yet ready for PR authors to take up label Jan 7, 2025
@kevjonesin
Copy link

Also when zooming line thickness doesn't scale relative to what's already been drawn.

I.e. if one leaves the line thickness slider in a fixed position and then makes marks at three different zoom levels the app will render lines with three different widths—relative to each other.

@naveensingh
Copy link
Member

Are you referring to #9?

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

No branches or pull requests

3 participants