On Sep 8, 2013, at 10:33 AM, Erwan Douaille <[email protected]> wrote:
> > > > 2013/9/8 <[email protected]> > Thanks Erwan. Really sweet tool. That video did the trick to stimulate my > interest to try it out, so I've got some feedback. Don't take them as > criticisms. They cover some things that probably have not been a priority so > far. > > feedbacks good or not is always needed :) > Video: > V1. Nice structure, short and simple. Easy to follow, except the text you > type in is hard to read. There is a lot of irrelevant space around the tool. > It would be better is you could zoom in, so the tool takes the entire frame. > You might also consider shrinking the size of the tool a bit from its default > size. The clearer the video the more people will be drawn in to try it out. > > Yeah, maybe i should add subtitles ... I will apply your tips for the next > videos. > Application: > A1. It would be good to have a 'Duplicate' icon with the widget halos. Once > you have sized a button, the easiest way to get another the same size is to > copy. Also in some cases, the code would be very similar. > > Ok, my mentor suggested me to create it :) > Added to todo list > > A2. With 'GenerateUI > Provide following information...' it is awkward that > Category defaults to a system category. As well, having all system categories > listed in the pulldown is awkward, since most of these will never be a > target. However offhand I can't think of a good solution. Some way to sort > probable targets near the top. Maybe a tree list where the top-level items > provide a filter for the second level items. For example, top-level items / > filters might be: > * Recent - in particularly, the category just created would appear here > * Categories ending in '-UI' > * Categories containing classes that inherit from a particular object (like > ComposableModel) > * Categories that contain classes that contain a certain method (like > #generatedFromUIPainter) > * 'All categories' would be the last as the last top-level items . > > Yeah. ComposableModel allsubclasses packagesList shoud be more efficient, > like you suggested. > Added to todo list > > > A3. In the same Category field, the hash(#) and the quotes('') seem > redundant. Indeed, when I click the add category <+> button and type > MyCounter then click <Add>, it shows in the list without quotes e.g. > #MyCounter instead of #'MyCounter' like the rest of them. If I try to create > 'MyCounter2' then I get #''MyCounter2''. > > Smalltalk environment organization categories sorted asArray inspect > ...holds #MyCounter and #'''MyCounter2''' instead of #'MyCounter' and > #'MyCounter2' like the rest of the entries, > so the problem would seem to be in UIPainterSpecGenerationCategoryAddUI >> > button > line: SystemOrganizer default addCategory: self input getText asString. > > For comparison, I looked at how Nautilus adds packages (by putting a halt at > the top of AbstractNautilusUI>>addPackage), which uses [self > browsedEnvironment packageOrganizer createPackageNamed: name] in > #addPackageBasedOn: . For one thing, this uses [self browsedEnvironment > packageOrganizer] evaluates to RPackageOrganiser. I don't know how that > differs from SystemOrganizer, but maybe consider doing it the same of > Nautilus. > > I know there is bugs here, my latest commit was to fix some bugs with it, but > it's not enough. I also have the problem of the refresh. > Thanks for the Nautilus tip, i will have a look on it > Added to todo list I think this behaviour is managed in AbstractTool. Maybe you could consider having an instance of AbstractTool as an inst var and delegate this job to it :) I just figured out that you will maybe need a way to specify the "browsedEnvironment" Ben > > > A4. It would be nice if there was an <Open> or <Test> button at the top, that > generates and opens the form, rather than having to do it manually from a > Workspace. > Yeah, maybe a window "open or not" after the generation process > Added to todo list > > > A5. Rather than provide the Class Name, Category and Title every time you > generate, these would be better as project properties that remain consistent > every time you generate. > Hum ... many changes for that i guess. > I don't promise anything but i will have a look on it :) > Added to todo list > > > A6. The tooltip of the 'paperclip' halo icon is 'unknown tooltip'. Btw, what > is its purpose. Clicking makes it turn red but otherwise seems to have no > effect. > The effect is "sticky morph" when you move/resize it, the morph will stick to > the closest point of the grid. When the button is red this option is disable. > Are you sure than it does not work ? It works for me :s > I also added to my todo list the ability to change the grid size. > > > cheers -ben > > Thanks for feedbacks :) > > Erwan Douaille wrote: >> >> Here is a video about how to build a counter with UIPainter: >> https://www.youtube.com/watch?v=S4IfW0TQhSc >> >> >> 2013/9/7 Erwan Douaille <[email protected]> >> >> >>> Hi :) >>> >>> Yes i will. I'm writting a documentation about how to install it, use it, >>> ... add new widgets ... >>> As a part of the GSoC i have to write doc, tests .... >>> >>> I will do some screencasts. A counter is a nice idea. >>> Tonight i will upload a screencast for the counter :) >>> >>> >>> 2013/9/6 Alexandre Bergel <[email protected]> >>> >>> >>>> Hi! >>>> >>>> I've just tried and it seems to work as expected. >>>> Can you please give a short tutorial (could be like 4 pictures) on how to >>>> create a counter: label initialized with 0, +1 and -1 button? >>>> >>>> Cheers, >>>> Alexandre >>>> >>>> >>>> >>>> On Sep 5, 2013, at 3:16 PM, Erwan Douaille <[email protected]> >>>> wrote: >>>> >>>> >>>>> Hi, >>>>> >>>>> The GSoC deadlines is 16 september. It means than i have one week to >>>>> >>>> fix bugs and implements some new features. The best way to find bugs is >>>> user feedbacks. >>>> >>>>> I will be glad to fix many bugs than you will find. (many bugs found in >>>>> >>>> 5 minutes with my mentor ...) >>>> >>>>> Here is instructions to load UIPainter : >>>>> >>>>> Gofer new >>>>> smalltalkhubUser: 'ErwanDouaille' project: 'UIPainter'; >>>>> configurationOf: 'UIPainter'; >>>>> load. >>>>> ConfigurationOfUIPainter load >>>>> >>>>> >>>>> Here is my blog: http://uipainter-gsoc.over-blog.com >>>>> >>>>> Thanks for testing, feedbacks and ideas :) >>>>> >>>>> -- >>>>> Best regards, >>>>> >>>>> Douaille Erwan <[email protected]> >>>>> >>>>> -- >>>>> You received this message because you are subscribed to the Google >>>>> >>>> Groups "Smalltalk GSoC students" group. >>>> >>>>> To unsubscribe from this group and stop receiving emails from it, send >>>>> >>>> an email to [email protected]. >>>> >>>>> For more options, visit https://groups.google.com/groups/opt_out. >>>>> >>>> -- >>>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>>> Alexandre Bergel http://www.bergel.eu >>>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>>> >>>> >>>> >>>> >>>> >>>> >>> -- >>> Best regards, >>> >>> Douaille Erwan <[email protected]> >>> >>> >> > > > > > > -- > Best regards, > > Douaille Erwan <[email protected]>
