Eyecandy::drawZoom should use drawCone

Bugs item #1850602, was opened at 2007-12-14 09:47
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=835077&aid=1850602&group_id=165310

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Benoit Jacob (benoitjacob)
Assigned to: Marcus D. Hanwell (cryosuk)
Summary: Eyecandy::drawZoom should use drawCone

Initial Comment:
Hey Marcus,

Since you’ve been working both on drawZoom and drawCone I guess you had that in mind already, so the following is written with this assumption; If you hadn’t planned to do this, just tell me and I’ll try doing it :slight_smile: that is, if you like it :slight_smile:

Currently the drawZoom arrow is two-headed:

<---->

If you draw the heads as cones, the cone pointing backwards won’t be shown because of backface culling, and even if you disable backface culling it will still have the normal vectors pointing in the wrong direction.

I would suggest not bothering about that and instead going for a single-headed arrow instead:

<—

with only one head pointing toward the camera.

Since that cone will always be pointing toward the camera, you don’t need to disable backface culling, and you don’t need to do two-pass rendering. Also, the base disc is unwanted, but there’s no problem here, as it’ll get backface-culled.


You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=835077&aid=1850602&group_id=165310