• Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
  • Asset Store
  • Get Unity

UNITY ACCOUNT

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account
  • Blog
  • Forums
  • Answers
  • Evangelists
  • User Groups
  • Beta Program
  • Advisory Panel

Navigation

  • Home
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
    • Blog
    • Forums
    • Answers
    • Evangelists
    • User Groups
    • Beta Program
    • Advisory Panel

Unity account

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account

Language

  • Chinese
  • Spanish
  • Japanese
  • Korean
  • Portuguese
  • Ask a question
  • Spaces
    • Default
    • Help Room
    • META
    • Moderators
    • Topics
    • Questions
    • Users
    • Badges
  • Home /
avatar image
1
Question by yeagz7 · Sep 25, 2013 at 04:35 AM · inspectorchildreneditorguipropertydrawerfoldout

Property Drawer and Children

Hi guys,

I'm trying to fake a dictionary in the inspector using an enum and an array. I got to the point where it looks (mostly) correct but when I override GetPropertyHeight it covers the foldout's child and I can't edit their values. Does anyone know a solution to this? Thanks in advance.

alt textren

 [CustomPropertyDrawer(typeof (NameListAttribute))]
 public class NameListDrawer : PropertyDrawer
 {
     private NameListAttribute nameListAttribute { get { return ((NameListAttribute) attribute); }}
     private bool foldout = false;
     private const float ITEMSIZE = 15f;
 
 
     public override float GetPropertyHeight(SerializedProperty property, GUIContent label)
     {
         if (foldout)
         {
             return base.GetPropertyHeight(property, label) + ITEMSIZE * Enum.GetNames(nameListAttribute.names).Length;
         }
         else
         {
             return base.GetPropertyHeight(property, label);
         }
     }
 
     public override void OnGUI(Rect position, SerializedProperty property, GUIContent label)
     {
         foldout = EditorGUI.Foldout(position, foldout, label);
         if (foldout)
         {
             string[] names = Enum.GetNames(nameListAttribute.names);
 
             for (int i = 0; i < names.Length; i++)
             {
                 Rect rect = EditorGUI.IndentedRect(position);
                 rect.y += 15*(i + 1);
                 rect.height = 15f;
                 EditorGUI.PropertyField(rect, property.GetArrayElementAtIndex(i), new GUIContent(names[i]));
             }
         }
     }
 }
propertydrawer bug.png (7.1 kB)
Comment
Add comment
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

3 Replies

· Add your reply
  • Sort: 
avatar image
2

Answer by OLP · Oct 02, 2013 at 06:41 PM

When overriding GetPropertyHeight, it sets the height property of your Rect position to the total height. All you have to do is reset that height before calling EditorGUI.Foldout.

 public override void OnGUI(Rect position, SerializedProperty property, GUIContent label)
 {
     position.height = 16f;
     foldout = EditorGUI.Foldout(position, foldout, label);
Comment
Add comment · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image
2

Answer by Deacon · Oct 31, 2013 at 05:28 PM

Just a side note, instead of using the foldout variable used the built in property.isExpanded variable to see if you have expanded the property you are working with. PropertyDrawers are not 1-to-1 with the properties they draw.

Comment
Add comment · Show 2 · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users
avatar image primitiveType · May 02, 2017 at 05:14 PM 1
Share

This helped me. Without this note, my drawers were working in such a way that if I had a list of instances of my class, expanding one instance expanded all of them. Looks like there is one drawer object that handles all instances of the class.

Thanks!

avatar image Deacon primitiveType · May 02, 2017 at 05:29 PM 0
Share

Glad to be of assistance.

avatar image
0

Answer by gardian06 · Sep 26, 2013 at 02:42 PM

you don't call GetPropertyHeight after foldout, and names are set (from what I have found if you want something really special to happen in GetPropertyHeight then you need to call it yourself in your PropertyDrawer as repainting is not done by default, or as often like with EditorWindows, or CustomInspectors

the other suggestion is instead of doing hard set increments of rect.y (and rect.height; why is this being set to 15 in every iteration of the for loop?)

 //just right before your forloop do:
 rect.height = GetPropertyHeight/names.Length;
     //inside the for loop 
     rect.y += rect.height
Comment
Add comment · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

Your answer

Hint: You can notify a user about this post by typing @username

Up to 2 attachments (including images) can be used with a maximum of 524.3 kB each and 1.0 MB total.

Welcome to Unity Answers

The best place to ask and answer questions about development with Unity.

To help users navigate the site we have posted a site navigation guide.

If you are a new user to Unity Answers, check out our FAQ for more information.

Make sure to check out our Knowledge Base for commonly asked Unity questions.

If you are a moderator, see our Moderator Guidelines page.

We are making improvements to UA, see the list of changes.



Follow this Question

Answers Answers and Comments

20 People are following this question.

avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image

Related Questions

EditorGUI.Foldout consumes click so GUI.Button doesnt work when inside of foldout region. 1 Answer

How to show type text in EditorGUI.ObjectField? 1 Answer

Inspector Overlapping Text Label at a Position 1 Answer

EditorGUI.Foldout -- Cannot interact with contents! 1 Answer

How can PropertyDrawer with fouldouts to draw at the appropriate height? 3 Answers

  • Anonymous
  • Sign in
  • Create
  • Ask a question
  • Spaces
  • Default
  • Help Room
  • META
  • Moderators
  • Explore
  • Topics
  • Questions
  • Users
  • Badges