'List' and 'Text' actions inconsistent, 'List' doesn't work as expected

[BillHeaton]

Prerequisites

• [x ] I have searched for similar issues in both open and closed tickets and cannot find a duplicate.
• [x ] The issue still exists against the latest master branch of yii2-tree-manager.
• [x ] This is not an usage question. I confirm having gone through and read the documentation and demos.
• [x ] This is not a general programming / coding question. (Those should be directed to the webtips Q & A forum).
• [x ] I have attempted to find the simplest possible steps to reproduce the issue.
• [ ] I have included a failing test as a pull request (Optional).

Steps to reproduce the issue

1. Reproducible on current demo

2. Select "Phones & Tables" on treeview

3. Select "Bell" under details icon and click Save

4. Notice missing icon for "Phones & Tablets"

5. Select "Text" "under Configure Icon As"

6. Click "Apply Demo Config"

7. Select "Phones & Tables on treeview

8. Notice missing icon for "Phone & Tablets"

9. Notice "4" on as "Icon"

10. Set "Icon" to "bell" and click save

11. Notice we have our bell.

12. Notice demo gets set back to list.

Expected behavior and actual behavior

When I follow those steps, I see...

I was expecting... Both options should be equivalent. On list the "4" the database and "fa fa-4" doesn't exist On text the "bell" is saved in the database and "fa fa-bell does exits

Environment

Bootstrap4 and 'fontAwesome' => true

Browsers

• [x ] Google Chrome
• [ ] Mozilla Firefox
• [ ] Internet Explorer
• [x ] Safari

Operating System

• [ ] Windows
• [x ] Mac OS X
• [ ] Linux
• [ ] Mobile

Libraries

• jQuery version:
• yii2-tree-manager version: Latest

Isolating the problem

• [x ] This bug happens on the demos page
• [x ] The bug happens consistently across all tested browsers
• [x ] This bug happens when using yii2-tree-manager without other plugins.

[kartik-v]

Resolved via above commit.