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 orientation of sensor rectangle in equatorial mode #3260

Merged
merged 1 commit into from
Jun 3, 2023

Conversation

10110111
Copy link
Contributor

@10110111 10110111 commented Jun 3, 2023

Now we get rid of the concept of polar angle inside Oculars::paintCCDBounds(), instead the correct orientation is automatically defined by the projector corresponding to the mount of the current telescope.

This should fix the wrong orientation of the rectangle in equatorial mode. Please check that it works in all relevant modes (some of which I might be unaware of).

Now we get rid of the concept of polar angle inside
Oculars::paintCCDBounds(), instead the correct orientation is
automatically defined by the projector corresponding to the mount of the
current telescope.
@github-actions github-actions bot requested review from alex-w and gzotti June 3, 2023 14:51
@alex-w alex-w added this to the 23.2 milestone Jun 3, 2023
@alex-w
Copy link
Member

alex-w commented Jun 3, 2023

The orientation is fixed, but the text in corners has density issue now on HiDPI devices

stellarium-002
stellarium-003

Copy link
Member

@gzotti gzotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for me, thanks. My approval depends on HiDPI fixes confirmed by @alex-w, of course.

@10110111
Copy link
Contributor Author

10110111 commented Jun 3, 2023

My approval depends on HiDPI fixes

Well, this is a separate problem orthogonal to this PR.

@10110111 10110111 merged commit e7b0f55 into Stellarium:master Jun 3, 2023
@10110111 10110111 deleted the ocular-rect-orientation branch June 3, 2023 16:18
@alex-w alex-w added the state: published The fix has been published for testing in weekly binary package label Jun 15, 2023
@github-actions
Copy link

Hello @10110111!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Jul 2, 2023
@github-actions
Copy link

github-actions bot commented Jul 2, 2023

Hello @10110111!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants