Hello Pierre! I did some usability review on your patch today... I think it is a great piece of work! If you are planing to do further developement, my suggestions would be:
- when clicking on a label with right mouse button, a popup should appear with at least two options "edit text" and "delete label", maybe something more if it makes any sense ... (copy to clipboard ? ) what currently happens is automatic opening of a textbox, which is a bit counter intuitive - show text of the label as a popup hint when mouse hovers for some amount of time over it - show the first line of label in status bar at the bottom as soon as the mouse goes over... last two could be implemented in similar way actually.. maybe reduced and extended hint or something... bye andraz On čet, 2006-01-19 at 00:48 +1030, Pierre Marc Dumuid wrote: > Hi Johannes / All, > > I've developed got a patch (attached) that adds text strings to the > labels, along with a listing ability in the assets window. Could you > possibly review it for me, (ensure that I've implemented all the delete > ing, and freeing of memory, and avoided race conditions. I'm not > completely comfortable with the patch, (kinda curious as to what happens > memory-wise if you right-click the label to edit it, then w/out closing > the "edit label text window", remove the label, then click "o.k" on the > "edit label text window" -- it doesn't crash, but I should probably > implement something?? any comments?? ) > > Pierre > > _______________________________________________ Cinelerra mailing list [email protected] https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra
