Editor inspector fails to display MRTK IconSet

[philip-lamb]

Describe the bug

When selecting a MRTK IconSet in the Unity Editor, the icons are not displayed. Instead an error is displayed in the inspector "You can't nest FoldoutHeaders, end it with EndHeaderFoldoutGroup.".

To reproduce

Steps to reproduce the behavior:

1. Install MRTK standard assets.
2. Navigate to default IconSet
3. Look at inspector pane

Expected behavior

Should see icons in set.

Screenshots

Your setup (please complete the following information)

• Unity 2020.2.2f1
• MRTK 2.5.4

Target platform (please complete the following information)

• HoloLens 2

Additional context

[atiemark]

Having the same issue. Found a tempory workound to add my own icon by modifying the DefaultIconSet.asset in an editor (create a new entry with the GUID of the icon you want the add)

[keveleigh]

Hmm, I'm not able to fully repro. Am I missing anything? I'm able to view the asset, but unable to fold out any of the foldouts due to packages being immutable:

[philip-lamb]

I re-imported the project (close project, delete library folder, reopen, set platform and build settings) and I can't reproduce this now. At a guess it could have been some out of date asmdef.

[MiPa12]

Hi, I'm having the same issue.

Your setup (please complete the following information) Unity 2020.3.11f1 MRTK 2.7.0

[keveleigh]

@RogPodge Was this an issue fixed by https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9999?

[RogPodge]

Yep! should be fixed

[daffy76]

same problem

[istvanContXtual]

Same problem,

Created a new icon set, Quad and Sprite Icons error in inspector panel: "You cant nest Foldout Headers, end it with EndHeatherFoldoutGroup"

[MilanPolle]

This problem still exists in MRTK 2.7.2

[RogPodge]

The fix is checked into the main repo but is not yet released in one of our official packages.

We are looking to put out a release which contains this bug fix before end of the year, so stay tuned

[MilanPolle]

Thank you for your reply! I downloaded the fix and applied it to the mrtk core package, this solved the problem.

[kkyuri0419]

@MilanPolle I am facing the same issue with MRTK 2.7.2 May I ask what do you mean by you downloaded the fix?

[MilanPolle]

@kkyuri0419 the file with the fix is here: https://raw.githubusercontent.com/RogPodge/MixedRealityToolkit-Unity/02f3e41f650a14eb744b41507270fea0d0470d6a/Assets/MRTK/SDK/Features/UX/Scripts/Buttons/ButtonIconSet.cs

Open the /packages/MixedReality/ folder, in your Unity project and then open the .toolkit.foundation tgz file there (with 7Zip, for instance).

In the tgz file, navigate to /package/SDK/Features/UX/Scripts/Buttons/ and there you can replace the ButtonIconSet.cs file with the one you've downloaded.

Unity will process the changes by itself and then you should be able to add your icons in the editor without the errors that were there.

[kkyuri0419]

@kkyuri0419 the file with the fix is here: https://raw.githubusercontent.com/RogPodge/MixedRealityToolkit-Unity/02f3e41f650a14eb744b41507270fea0d0470d6a/Assets/MRTK/SDK/Features/UX/Scripts/Buttons/ButtonIconSet.cs

Open the /packages/MixedReality/ folder, in your Unity project and then open the .toolkit.foundation tgz file there (with 7Zip, for instance).

In the tgz file, navigate to /package/SDK/Features/UX/Scripts/Buttons/ and there you can replace the ButtonIconSet.cs file with the one you've downloaded.

Unity will process the changes by itself and then you should be able to add your icons in the editor without the errors that were there.

Thank you so much @MilanPolle !! it perfectly works!

[MilanPolle]

@kkyuri0419 Glad to hear that! Thanks for letting me know.

[jbienz]

Just curious, since this was fixed in March what caused it to be missed in the 2.7.2 release? Also, would love to see this fixed in the next package release. It's kind of painful to apply the hotfix above and the next MRFT update will likely overwrite it. Thanks!

[keveleigh]

Just curious, since this was fixed in March what caused it to be missed in the 2.7.2 release?

The PR that fixed this was #9999, which was merged in June and just missed the 2.7.2 stabilization cutoff. It'll definitely be in our next release!

[dvDevel]

You may fix the bug with the link. The easiest way is to update to 2.7.3 fundamental with MixedRealityFeatureTool. In my case, nothing appears any errors after the update to 2.7.3.

Additionally. I tried exchanging Button IconSet.cs into tar at the package and tied it again. but It's not working.