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

Additional advanced options for ping & name plates #59

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

Outshynd
Copy link
Contributor

I had wanted to add an option for Fontaine's FOV Fixes mod users to be able to still display name plates and pings when aiming with an optic scope, but to not apply the optic's zoom&FOV to the location of the name plate / ping sprite. To that end, I added a boolean toggle for each as an 'Advanced' option, not normally visible.

Also added parts of a feature request from ArcticFox037 on Discord as advanced options: minimum ping opacity and a toggle to not scale ping based on distance.

No breaking changes, defaults are set up so that anyone who does not look in the Advanced settings of Configuration Manager will not even notice a difference.

@Lacyway
Copy link
Member

Lacyway commented May 26, 2024

Do you have a 3.9.0 build you can use to do this and retarget?
I've kind of halted development on 3.8.3 unless critical bugfixes are needed.

3.9.0 seems like it's not far off and it has big improvements so I decided to focus development on it instead.

@Outshynd
Copy link
Contributor Author

I don't have a 3.9.0 build, no, sorry. These are pretty surface level changes, I'm sure they can integrate straight in to 3.9.0, I just haven't set that up, nor do I have plans to until my group switches to 3.9.0 as stable. No big deal, I can re-implement later or whatever works.

@Lacyway
Copy link
Member

Lacyway commented May 26, 2024

No worries, leave the PR and I'll migrate it to 3.9.0 next time I work on it.

@Lacyway Lacyway changed the base branch from main to 3.9-dev May 30, 2024 06:13
@Lacyway Lacyway self-assigned this May 30, 2024
@Lacyway Lacyway self-requested a review May 30, 2024 06:13
@Lacyway
Copy link
Member

Lacyway commented May 30, 2024

Will try to merge this today.

@Lacyway Lacyway merged commit 02ee282 into project-fika:3.9-dev Jun 3, 2024
1 check failed
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