26.05.2011, 21:02, "Geoffrey Hutchison" <[email protected]>:
>>  It needs to be implemented but I don't see any difficulties. Will look on 
>> it when I have some free time.
>
> Great, thanks!
>
>>  However there may be some troubles with supporting this feature for both 
>> atoms and bonds (what should be done for "mixed" selection?)
>
> I think for most people, they care about the atoms rather than bonds in a 
> "mixed" selection. I guess the best solution would be a dialog with tabs for 
> atoms or bonds, but that's probably better left as a ToDo for now.

https://github.com/annulen/avogadro/commit/a69c045f2f1a192b166a954472dc15d4535c06fd

Probably it should be further polished to reduce code duplication

Also, maybe it's a good time to fix custom property manipulations. 
* We discussed use of QProperties to reduce memory overhead
* I don't like how access to colors is implemented - first I need to check 
custom color, than color map, but it would be much more pleasant to have 
something like Atom::color() returning actual property value.

-- 
Regards,
Konstantin

------------------------------------------------------------------------------
vRanger cuts backup time in half-while increasing security.
With the market-leading solution for virtual backup and recovery, 
you get blazing-fast, flexible, and affordable data protection.
Download your free trial now. 
http://p.sf.net/sfu/quest-d2dcopy1
_______________________________________________
Avogadro-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/avogadro-devel

Reply via email to