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

CLL-310: Do not allocate space for crosshair label is crosshair is di… #1743

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

eugene-korobko
Copy link
Contributor

@eugene-korobko eugene-korobko commented Nov 28, 2024

Improve price scale width calculation

Type of PR: bugfix / enhancement
PR checklist:

  • Addresses an existing issue: fixes #
  • Includes tests
  • Documentation update

Overview of change:
If crosshair is disabled by options, we do not need to preallocate space for all the possible crosshair labels within the price scales

@eugene-korobko eugene-korobko force-pushed the CLL-310-price-axis-width-without-crosshair branch from 560b75a to 4096374 Compare November 28, 2024 06:30
@SlicedSilver SlicedSilver self-requested a review December 2, 2024 09:53
@SlicedSilver SlicedSilver merged commit ad7c919 into master Dec 2, 2024
15 of 21 checks passed
@SlicedSilver SlicedSilver deleted the CLL-310-price-axis-width-without-crosshair branch December 2, 2024 09:54
SlicedSilver added a commit that referenced this pull request Dec 2, 2024
…thout-crosshair

CLL-310: Do not allocate space for crosshair label is crosshair is di…
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