Skip to content

Conversation

@sunzenshen
Copy link

Description

Provide settings toggle for previewing dynamic accuracy

Toolchain

  • Windows MSVC VS2022

Linked Issues

@sunzenshen sunzenshen changed the base branch from master to 235_dynamicCrosshair July 12, 2025 21:41
@sunzenshen
Copy link
Author

sunzenshen commented Jul 12, 2025

@sunzenshen
Copy link
Author

Separated setting from networked crosshair based on this feedback:

I don't think it makes sense to save and network whether to preview the dynamic crosshair or not with the rest of the crosshair settings

@sunzenshen
Copy link
Author

In response to feedback:

The way it looks currently, it's behavior seems unintuitive compared to how it's laid out
I know it's gonna be very tricky but a button with "test crosshair" text would fit this behavior better I think

@sunzenshen
Copy link
Author

In response to:

Put it in the same row as the import/export crosshair see how that goes

@AdamTadeusz
Copy link
Owner

looks great, I've merged it to resolve conflicts and shortened the text in the button a bit, it's getting crowded with the new reset to default button

@AdamTadeusz AdamTadeusz merged commit 9cc82d8 into AdamTadeusz:235_dynamicCrosshair Jul 31, 2025
@sunzenshen sunzenshen deleted the toggle-dynamic-accuracy-preview branch August 1, 2025 03:30
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