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

Fix/override button style #225

Merged
merged 6 commits into from
Feb 12, 2025
Merged

Conversation

lukengda
Copy link
Contributor

Herby, i propose the following changes to solve #224. While we're at it, i've improved and added a bit to the tests.

I have taken the liberty of adding a version increment if a release of this fix is being considered. Let me know if you need something to get this going. Thank you very much!

Lukas Engelter and others added 5 commits February 12, 2025 08:36
TextButton style merge is implemented in a way that the original object takes precedence over the argument.
We therefore switch the merge orientation to ensure that it is possible to  override every style.

Closes Pyozer#224
@lukengda
Copy link
Contributor Author

i was able to reproduce (and fix) the build error once i've upgraded my local flutter version to stable.

…support current flutter / dart version

Update Android build tooling:
- Bump Android Gradle Plugin from 7.3.0 to 8.1.0
- Upgrade Gradle wrapper from 7.6.3 to 8.0
@lukengda lukengda force-pushed the fix/override-button-style branch from 8fd0bfb to f758b55 Compare February 12, 2025 10:32
@lukengda
Copy link
Contributor Author

it seems to be working now, Java 17 and flutter upgrade did the job.

i was able to reproduce (and fix) the build error once i've upgraded my local flutter version to stable.

@Pyozer Pyozer merged commit 75d4d14 into Pyozer:master Feb 12, 2025
2 checks passed
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