Summary: Using specified FOV's for each planet so the user can zoom. Adding a button to select FOV from the KStars FOV list. Adding another decimal to FOV for small moons.
ClosedPublic

Authored by lancaster on Aug 24 2018, 5:10 AM.

Diff Detail

Repository
R321 KStars
Branch
xplanetFOV
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2200
Build 2218: arc lint + arc unit
lancaster created this revision.Aug 24 2018, 5:10 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptAug 24 2018, 5:10 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
lancaster requested review of this revision.Aug 24 2018, 5:10 AM
mutlaqja requested changes to this revision.Aug 24 2018, 8:59 AM
mutlaqja added inline comments.
kstars/auxiliary/xplanetimageviewer.cpp
313 ↗(On Diff #40348)

"Zoom to a specific FOV. This has no effect when hovering over an object"

1064 ↗(On Diff #40348)

for(const FOV *f : KStarsData::Instance()->getAvailableFOVs())

This revision now requires changes to proceed.Aug 24 2018, 8:59 AM
lancaster updated this revision to Diff 40371.Aug 24 2018, 3:02 PM
  • Fixing text

Updating D15050: Summary:

Using specified FOV's for each planet so the user can zoom.
Adding a button to select FOV from the KStars FOV list.
Adding another decimal to FOV for small moons.

lancaster marked 2 inline comments as done.Aug 24 2018, 3:03 PM
mutlaqja added inline comments.Aug 24 2018, 4:48 PM
kstars/auxiliary/xplanetimageviewer.cpp
1063

for, not foreach

lancaster updated this revision to Diff 40378.Aug 24 2018, 7:49 PM
  • Using for not foreach

Updating D15050: Summary:

Using specified FOV's for each planet so the user can zoom.
Adding a button to select FOV from the KStars FOV list.
Adding another decimal to FOV for small moons.

mutlaqja accepted this revision.Aug 25 2018, 7:15 AM
This revision is now accepted and ready to land.Aug 25 2018, 7:15 AM
This revision was automatically updated to reflect the committed changes.