Uploaded image for project: 'Qt 3D Studio'
  1. Qt 3D Studio
  2. QT3DS-4034

Q3DSSceneElement::setCurrentSlideIndex sets index incorrectly

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Not Evaluated
    • Qt 3D Studio 2.6
    • Qt 3D Studio 2.5
    • OpenGL Runtime
    • None
    • 8054702ab93161ba509e5dfed4a893cca62c2481

    Description

      Incoming index is incremented by one for no apparent reason.

      As this is a setter function, following sequence meant to increment slide by one works incorrectly: setCurrentSlideIndex(currentSlideIndex() + 1));

      Not sure if changing this breaks any existing content, but the current functionality is not consistent.

      edit: SceneElement API indexing still has different slide indexing compared to Q3DSPresentation. This is confusing when receiving slideEntered/Exited notifications from both SceneElement and Presentation, as numbers are off by one.

      I hesitate changing the indexing of signaling from Q3DSPresentation (signals might be used internally in Runtime), and rather change SceneElement API to follow convention where slide 1 corresponds to index 1 (unlike in qt3d where first slide was index zero).

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              jakangas Janne Kangas
              jakangas Janne Kangas
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes