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

possible fix for opacity widget causing hairlines and possible flicker #1169

Merged
merged 1 commit into from Apr 10, 2022

Conversation

ibrierley
Copy link
Collaborator

@ibrierley ibrierley commented Feb 20, 2022

This is a possible workaround for the flicker and hairline issues #1126 & #1156

It removes the Opacity widget (I'm not sure this is going to get fixed easily by Flutter, so think we will have to create our own), and introduces the opacity to the RawImage instead.

This feels possibly logical, however, I'm not confident on the Animated Tile stuff, so this may not be exactly the same.... I assume the opacity at this position is for fading in new tilesets/zooms ? If anyone knows this part of the code, it would be useful to give some thoughts, as I'm a little unsure how to test it properly.

It would also be useful for those with the initial problems to see if it helps.

Rotation needs to be disabled for the original hairline type issues to appear, i.e interactiveFlags: ~InteractiveFlag.rotate in MapOptions

@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@JaffaKetchup
Copy link
Member

I'll try to test soon

Copy link
Member

@JaffaKetchup JaffaKetchup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ibrierley
Unfortunately I can't test this. I've tried running the latest 'master' branch example app on the Home page, and I just can't see the white lines that I've seen before. Therefore I wouldn't be able to know if this has done anything?

Maybe this has fixed itself in Flutter updates? Alternatively (and I'm not sure why this would make a difference), I had to update the compileSdkVersion & targetSdkVersion to '31', and the ext.kotlin_version to '1.6.20'?

Can you try on your side, to see if you still get the issue?

@ibrierley
Copy link
Collaborator Author

Did you try with rotation disabled ? i.e interactiveFlags: InteractiveFlag.all & ~InteractiveFlag.rotate in MapOptions.

@JaffaKetchup
Copy link
Member

Ah no probably not - I just used the main example. My mistake: I'll try again tomorrow :)

@stx
Copy link

stx commented Apr 8, 2022

I've tested this and it works great. Nice work @ibrierley!

@JaffaKetchup JaffaKetchup self-requested a review April 10, 2022 12:36
Copy link
Member

@JaffaKetchup JaffaKetchup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Seems to fix the issue, but not sure what knock on impacts this might have. Will merge now...

@JaffaKetchup JaffaKetchup merged commit 3ead644 into fleaflet:master Apr 10, 2022
@JaffaKetchup JaffaKetchup linked an issue May 8, 2022 that may be closed by this pull request
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.

Flicker grid between map tiles on zoom
3 participants