bug in UIInput

[jmclaren]

When you type more characters into a Input widget than the lineWidth of the child UILabel with single line, centre pivot, the whole UILabel teleports off to the right. observed in 2.3.1 and 2.3.3h

[ArenMook]

That's because only left-aligned input fields are supported. This is enforced by the input creation process -- when you create a new input field, it's set to be left-aligned. You've had to have changed it to be center-aligned yourself. Change it back.

[jmclaren]

We would prefer to be able to use centre alignment too, is this possible for a future release?

[ArenMook]

Perhaps in the future, sure. You're welcome to submit a patch for it!

[ryan]

1. diff NGUI-2.5.1/Assets/NGUI/Scripts/UI/UIInput.cs Project/Assets/NGUI/Scripts/UI/UIInput.cs
2. 468c468,483
3. <
4. ---
5. >                                       else if (mPivot == UIWidget.Pivot.Center)
6. >                                       {
7. >                                               label.pivot = UIWidget.Pivot.Right;
8. >                                               pos.x -= label.lineWidth * 0.5f;
9. >                                       }
10. >                                       else if (mPivot == UIWidget.Pivot.Top)
11. >                                       {
12. >                                               label.pivot = UIWidget.Pivot.TopRight;
13. >                                               pos.x -= label.lineWidth * 0.5f;
14. >                                       }
15. >                                       else if (mPivot == UIWidget.Pivot.Bottom)
16. >                                       {
17. >                                               label.pivot = UIWidget.Pivot.BottomRight;
18. >                                               pos.x -= label.lineWidth * 0.5f;
19. >                                       }
20.  

I realized after making this change that right-aligned labels aren't supported either, or I probably would have made this slightly more elegant.  Silly me.  Left that as an exercise for the reader.

[ArenMook]

This also assumes lineWidth was set.

[ryan]

So does the existing code just above where that's added. 

[ArenMook]

True!