[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-22 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1050 Will merge if there's no more discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-22 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1050 Tested, LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-21 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1050 @corneadoug How about it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wi

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1050 Would be better in a separated file --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature en

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1050 @corneadoug Thanks for the quick reply. I've googled `orderBy` and tried it first but this was not applied by map as you told me. I'll try to do with custom filter with keys. I have another quest

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-20 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1050 @jongyoul seems to be working, the gracious way would have been to use an orderBy filter (https://docs.angularjs.org/api/ng/filter/orderBy), but it only applies to Array, and we are using an Ob

[GitHub] zeppelin issue #1050: [MINOR] Show properties ordered alphabetically

2016-06-20 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1050 @corneadoug Could you please review this PR? It has very simple change but I don't know where the right position of that function is. --- If your project is set up for it, you can reply to this