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

[Style] Addition of the support of background-image-repeat #42

Merged
merged 3 commits into from
Apr 11, 2023

Conversation

tfabretti
Copy link

Hi again ! :D
I thought I could delete the branch and add another to the pull request bu it turns out I can't, so i recreated the PR #40 (linked to issue #39) :P
Sorry, I'm still struggling with Git and Github :(

@ianharrigan
Copy link
Member

Still not sure its right im afraid... in your PR there is code being added for "drawRoundedRect", this was removed and your PR is re-adding it, which makes me cautious about what else it might be re-adding. If i were you it might make sense to close this PR, and make sure you have the most up to date version of haxeui-heaps (which presumably you dont currently), and then "redo" your changes. PITA for sure though... :/

@tfabretti
Copy link
Author

Ah ! I made a bad merge, at some point. Then, I didn't check closely enough. Since it was after my new function, I missed it being an entirely other function ^^'

@tfabretti
Copy link
Author

I've fixed the merging to make sure I didn't erase some new stuff, it should be alright, now :D

@ianharrigan ianharrigan merged commit e1705f1 into haxeui:master Apr 11, 2023
@ianharrigan
Copy link
Member

Great, thanks for this! :)

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.

2 participants