Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge tern def/plugin selection in the same treeview #63

Closed
angelozerr opened this issue May 11, 2014 · 15 comments
Closed

Merge tern def/plugin selection in the same treeview #63

angelozerr opened this issue May 11, 2014 · 15 comments

Comments

@angelozerr
Copy link
Owner

Today, we have 2 treeviews for tern definition and tern plugin. I think it should be better to merge those treeview in one treeview.

What do you think about that?

Which names could we use? Tern Plugins? Tern facets? Tern features? Tern capabilities?

@angelozerr
Copy link
Owner Author

@vrubezhny @dgolovin @mickaelistria @maxandersen @pascalleclercq @gamerson @paulvi could you please give me your comment about this issue, because I think it's important to know your opinion about the new UI to select tern defs/plugins.

Thank's!

@mickaelistria
Copy link
Contributor

I've not used Tern so I'm just guessing, but I indeed think that a single treeviewer for those 2 items seem better that 2. Indeed, a user who wants to "hack" is tern configuration would most likely want a single entry point.
If you need to name a view, I'd simply call it "Tern".

@angelozerr
Copy link
Owner Author

Thank's a lot @mickaelistria for your answer. I will do that.

Today we have in the project preferences :

Tern 
 -> Plugins
 -> Type Definitions

to select JSON Type Definitions (simple JSON) and Tern plugin (javascript which uses JSON Type Definitions)

I wil remove Type Definitions entry to merge Plugins+Type Definitions :

Tern 
 -> Plugins

If somebody wishes other label than "Plugins", don't hesitate to give me a suggestion.

Thank's!

@angelozerr
Copy link
Owner Author

For your info, I have finished this issue. I have decided to use "Facets" instead of using "Plugins" :

  • Facet is used to define "Plugins" and "JSON Type Definitions"
  • Facet is used too for "Project Facets" which gives some features like Java, Javascript, CXF, etc

Here a screenshot of the new UI Tern Facet project property to select "Plugins" and "JSON Types Definitions" :

ternfacets

You can notice too that there are now icons for each "Plugins" and "JSON Types Definitions"

Hope you will like it.

@gamerson
Copy link
Contributor

Looks nice... if you wanted to avoid the word "facets" because that is used
in the faceted project framework already, what if you called it "Tern >
Extensions"?

On Wed, May 21, 2014 at 3:30 PM, Angelo notifications@github.com wrote:

For your info, I have finished this issue. I have decided to use "Facets"
instead of using "Plugins" :

  • Facet is used to define "Plugins" and "JSON Type Definitions"
  • Facet is used too for "Project Facets" which gives some features
    like Java, Javascript, CXF, etc

Here a screenshot of the new UI Tern Facet project property to select
"Plugins" and "JSON Types Definitions" :

[image: ternfacets]https://cloud.githubusercontent.com/assets/1932211/3037184/95fa6f26-e0b9-11e3-98a5-994eaf221b2a.png

You can notice too that there are now icons for each "Plugins" and "JSON
Types Definitions"

Hope you will like it.


Reply to this email directly or view it on GitHubhttps://github.com//issues/63#issuecomment-43721494
.

Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com

@mickaelistria
Copy link
Contributor

I'm somehow surprised to see that there is no "JavaScript" top category.
Tern and Angular could fit in.
Don't you depend on JSDT ?

@angelozerr
Copy link
Owner Author

you called it "Tern >Extensions"?

I prefer using "Facet" instead of using "Extension", because Eclipse user are famliliar with "Facet" (you add some capabilities to your project). But if you prefer "Extension", I will change it.

I'm somehow surprised to see that there is no "JavaScript" top category.

"JavaScript" top category is ecma5 JSON Type Definition, I would like to avoidd changing the original name.

Tern and Angular could fit in.

Tern is the inference Javascript engine which is configured with plugins, JSON Type Definition. AngularJS Eclipse is based on the Tern plugin angular.

Don't you depend on JSDT ?
Yes and no:)

Tern IDE UI doesn't depends on JSDT. So you could use tern.java for other editor kind.

The project https://github.com/angelozerr/tern.java/tree/master/eclipse/tern.eclipse.ide.jsdt is a bridge between Tern and JSDT : it defines for instance compeltion extension for JSDT with tern.

@gamerson
Copy link
Contributor

I don't have a strong preference for extension over facet... both can work
for me.

On Wed, May 21, 2014 at 4:18 PM, Angelo notifications@github.com wrote:

you called it "Tern >Extensions"?

I prefer using "Facet" instead of using "Extension", because Eclipse user
are famliliar with "Facet" (you add some capabilities to your project). But
if you prefer "Extension", I will change it.

I'm somehow surprised to see that there is no "JavaScript" top category.

"JavaScript" top category is ecma5 JSON Type Definition, I would like to
avoidd changing the original name.

Tern and Angular could fit in.

Tern is the inference Javascript engine which is configured with plugins,
JSON Type Definition. AngularJS Eclipse is based on the Tern plugin angular.

Don't you depend on JSDT ?
Yes and no:)

Tern IDE UI doesn't depends on JSDT. So you could use tern.java for
other editor kind.

The project
https://github.com/angelozerr/tern.java/tree/master/eclipse/tern.eclipse.ide.jsdtis a bridge between Tern and JSDT : it defines for instance compeltion
extension for JSDT with tern.


Reply to this email directly or view it on GitHubhttps://github.com//issues/63#issuecomment-43726305
.

Greg Amerson
Liferay Developer Tools
Liferay, Inc. www.liferay.com

@angelozerr
Copy link
Owner Author

Ok thank's @gamerson I will keep "Facets" for the moment.

Just for your information, I have the intention to improve this UI with:

@maxandersen
Copy link

I agree with @gamerson i think facet is a very different thing than this.

This is something you would configure on a tern/javascript facet - it is not a facet itself.

@angelozerr
Copy link
Owner Author

Ok @maxandersen, we could rename facet to

  • modules?
  • add-ons?
  • other?

I would like to avoid using "plugin" because tern provides Tern plugin and JSON Type Definition.

My preference is module. I'm waiting for your answer.

@angelozerr
Copy link
Owner Author

@vrubezhny @dgolovin @mickaelistria @maxandersen @pascalleclercq @gamerson @paulvi are you OK to rename facet to module?

@vrubezhny
Copy link
Contributor

Personally me - yes, I'm OK with 'module' instead of 'facet'.

@angelozerr
Copy link
Owner Author

Ok I have created a new issue for that. See #81 I will try to do that this WE. If you don't like 'module', please tell me.

@angelozerr
Copy link
Owner Author

Ok changes was done (screenshot of wiki too). Now module is used instead of facet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants