Tasharen Entertainment Forum

Support => NGUI 3 Support => Topic started by: Eugene on February 15, 2017, 01:45:56 PM

Title: Texture/Sprite Corners visibility bug
Post by: Eugene on February 15, 2017, 01:45:56 PM
I believe it's not the way it's supposed to work:
http://clip2net.com/s/3Hyx7He

The problem place is (UIBasicSprite::981).
This bit:
  1. if ((y == 0 && bottomType != AdvancedType.Invisible) || (y == 2 && topType != AdvancedType.Invisible) ||
  2.       (x == 0 && leftType != AdvancedType.Invisible) || (x == 2 && rightType != AdvancedType.Invisible))
Should be changed to this:
  1. bool show = true;
  2.         if (y == 0 && bottomType == AdvancedType.Invisible)
  3.             show = false;
  4.         if (y == 2 && topType == AdvancedType.Invisible)
  5.             show = false;
  6.         if (x == 0 && leftType == AdvancedType.Invisible)
  7.             show = false;
  8.         if (x == 2 && rightType == AdvancedType.Invisible)
  9.             show = false;
  10.  
  11.                     if (show)
  12.      {

And then it should work as it should. Please, add it to next NGUI patch.
Cheers.
Title: Re: Texture/Sprite Corners visibility bug
Post by: mdeletrain on February 16, 2017, 05:58:31 AM
I may be wrong but I think this is the same as :

  1.     if ((y != 0 || bottomType != AdvancedType.Invisible) && (y != 2 || topType != AdvancedType.Invisible) &&
  2.           (x != 0 || leftType != AdvancedType.Invisible) && (x != 2 || rightType != AdvancedType.Invisible))

So to keep initial form... but it's less readable.
Title: Re: Texture/Sprite Corners visibility bug
Post by: ArenMook on February 18, 2017, 07:04:19 PM
I think you mean...
  1.                                         if (y == 0 && bottomType == AdvancedType.Invisible) continue;
  2.                                         if (y == 2 && topType == AdvancedType.Invisible) continue;
  3.                                         if (x == 0 && leftType == AdvancedType.Invisible) continue;
  4.                                         if (x == 2 && rightType == AdvancedType.Invisible) continue;
  5.  
  6.                                         Fill(verts, uvs, cols,
  7.                                                 mTempPos[x].x, mTempPos[x2].x,
  8.                                                 mTempPos[y].y, mTempPos[y2].y,
  9.                                                 mTempUVs[x].x, mTempUVs[x2].x,
  10.                                                 mTempUVs[y].y, mTempUVs[y2].y, c);
I've ran into this before myself and always thought to myself, "well it has been this way since the beginning... why change it now?". I can see how the current approach can cause some confusion though, seeing as there is no way to explicitly hide corners.
Title: Re: Texture/Sprite Corners visibility bug
Post by: Eugene on February 19, 2017, 01:15:04 PM
@ArenMook, yeah. As long as I can keep ngui changes in my project to the minimum, it works for me. Thanks!