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

Qt: Fix some Post Process Configuration Widget issues #2 #11866

Merged
merged 1 commit into from Jun 1, 2023

Conversation

Filoppi
Copy link
Contributor

@Filoppi Filoppi commented May 31, 2023

Same fix as #11846 but applied to int sliders instead of float ones

@Filoppi Filoppi marked this pull request as ready for review May 31, 2023 22:04
@Filoppi Filoppi marked this pull request as draft May 31, 2023 22:05
@AdmiralCurtiss
Copy link
Contributor

Is there a reason this is a draft? This looks fine to me, lint failure aside.

@Filoppi
Copy link
Contributor Author

Filoppi commented Jun 1, 2023

Is there a reason this is a draft? This looks fine to me, lint failure aside.

Because of lint. I was going to fix it later but I'm happy for you to do it if you want.

Same fix as dolphin-emu#11846 but applied to int sliders instead of float ones
@AdmiralCurtiss AdmiralCurtiss marked this pull request as ready for review June 1, 2023 19:39
@AdmiralCurtiss AdmiralCurtiss merged commit 744a5dd into dolphin-emu:master Jun 1, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants