Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [recommenders-dev] [GSOC 2013] SnipEditor Weekly Report

Hi Stefan,

On Sep 1, 2013, at 10:11 AM, Stefan Prisca <stefan.prisca@xxxxxxxxx> wrote:


Hi Marcel,
   
 
-> Set up a wizard to create new snippets.
Will this be part of the next week's sprint?
 
If you refer to the wizard here, i thought to implement it after i finish the metadata page. Anyway, I'll try to finish it as well until we merge the projects.

Makes sense to start the wizard later. Added it to the backlog.

the imp prefix is not hard coded, right? From the grammar documentation it seems to me that this isn't needed/can be blank.

I will make it optional.

Thanks. Added it to the backlog.

 
-> I created a sketch for the metadata page of the editor. It only shows some basic info about the snippet at this moment.
Where can we find it?

This is the other page of the editor. I referred to it as a sketch as it is not finished yet (maybe this was a bad choice of words). I need to add the other info on the page, make the buttons functional and make it a little more pleasing to the eye.

I found it in the "Test" branch :-) I've a couple of comments here.

Some of the old snippet metadata does not make much sense after we moved to JFace template language. For instance, specifying the snippet parameters doesn't make sense anymore since the template language already specifies variables and their respective type. This section can be removed.

The "Matching Patterns" was initially thought to be used to specify alternative search phrases people may use to find snippets. I'd say, we should rename this to "Search Phrases". Madhuranga's indexer should then be updated to support this phrases too.

"Summary" should be renamed to "Description" as it's essentially exactly that.

In addition, the editor should have some additional fields like "Author", "Tags", and a (read-only) field "Referenced Types" that lists all types referenced in the code snippet. The referenced types are highly relevant and helpful search terms. So they should be indexed too (by Madhuranga later on :-) ).

Regarding the layout:
I'd prefer to have the "Add…", "Edit", "Remove" buttons on the right after the list viewer. This is more "Eclipse style".

We may add a couple more fields to the metadata later on like ratings, comments etc. But for not, I'd stick with the ones mentioned above. They would make a great start.

Thanks,
Marcel





Best Regards,
Stefan.

_______________________________________________
recommenders-dev mailing list
recommenders-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/recommenders-dev


Back to the top