<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://jogamp.org/bugzilla/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.2"
          urlbase="https://jogamp.org/bugzilla/"
          
          maintainer="sgothel@jausoft.com"
>

    <bug>
          <bug_id>1454</bug_id>
          
          <creation_ts>2023-09-23 20:37:46 +0200</creation_ts>
          <short_desc>GraphUI Scene: Elevate active (selected) shape (add z-offset) and select (pick) in Z descending order</short_desc>
          <delta_ts>2023-09-30 01:48:15 +0200</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>3</classification_id>
          <classification>JogAmp</classification>
          <product>GraphUI</product>
          <component>Core</component>
          <version>2.6.0</version>
          <rep_platform>All</rep_platform>
          <op_sys>all</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P4</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>1458</dependson>
          <blocked>1464</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Sven Gothel">sgothel</reporter>
          <assigned_to name="Sven Gothel">sgothel</assigned_to>
          
          
          <cf_type>FEATURE</cf_type>
          <cf_scm_refs>84b26f9efcd62cc8c41bf3cd867482080d16d7a0
9f900c7f88d70808e24ed8b48087126dad0c3188
4b8574c63e100f0ef8bb2ad292d71f612e6cfceb</cf_scm_refs>
          <cf_workaround>---</cf_workaround>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>6863</commentid>
    <comment_count>0</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-09-23 20:37:46 +0200</bug_when>
    <thetext>Picking (select) a shape shall process all shapes in Z descending order, 
i.e. top shape first. This is a bug currently.

Note: Picking (selecting) a shape using a (mouse-)pointer device
is active by mouse-moved and not only mouse-clicked.
Therefor, we select shapes by mouse-over.

The active selected shape shall have an elevated Z offset to be rendered on top
of the others on same plane.
- This avoids them being rendered below others while moving them around etc.
- This also avoids flickering of overlapping shapes with mouse over.
- This stabilizes the UI experience</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>6866</commentid>
    <comment_count>1</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-09-23 20:45:16 +0200</bug_when>
    <thetext>Feature added w/ commit 84b26f9efcd62cc8c41bf3cd867482080d16d7a0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>6876</commentid>
    <comment_count>2</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-09-24 01:51:53 +0200</bug_when>
    <thetext>Resolution uses a multiple of current Z-Epsilon, see Bug 1458</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>6879</commentid>
    <comment_count>3</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-09-24 02:40:26 +0200</bug_when>
    <thetext>9f900c7f88d70808e24ed8b48087126dad0c3188
Bug 1454 - GraphUI Scene: Make active ZOffset scale configurable</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>6902</commentid>
    <comment_count>4</comment_count>
    <who name="Sven Gothel">sgothel</who>
    <bug_when>2023-09-30 01:48:15 +0200</bug_when>
    <thetext>commit 4b8574c63e100f0ef8bb2ad292d71f612e6cfceb

    Bug 1454 + Bug 1464: Maintain a Shape local &apos;zOffset&apos; and only consider it for sorting w/o actually modifying the position (enogh to be painted on top and for selection)
    
    Also use a simplified comparison using only float relational operations w/o NaN/Inf bit-stuff or epsilon,
    as it should be accurate enough for this cause.
    
    This shall also resolve Bug 1454, as we no more modify the position directly
    but the local zOffset field .. but this has to be seen (data race).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>