Hi Sergiu,

sdumitriu (SVN) wrote:
> Author: sdumitriu
> Date: 2009-04-19 21:40:15 +0200 (Sun, 19 Apr 2009)
> New Revision: 18827
>
> Added:
>    platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/boolean.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/calendar.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/database.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/groups.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/move.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/number.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/password.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/propertyType-Generic.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/rights.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/staticlist.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/string.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/textarea.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/tree.png
>    
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/users.png
>    platform/web/trunk/standard/src/main/webapp/resources/js/xwiki/dataeditors/
>    
> platform/web/trunk/standard/src/main/webapp/resources/js/xwiki/dataeditors/dataeditors.css
>    
> platform/web/trunk/standard/src/main/webapp/resources/js/xwiki/dataeditors/dataeditors.js
> Modified:
>    platform/web/trunk/standard/src/main/webapp/templates/editclass.vm
>    platform/web/trunk/standard/src/main/webapp/templates/editobject.vm
>    platform/web/trunk/standard/src/main/webapp/templates/editpanelsvars.vm
> Log:
> XWIKI-3635: Improve the object and class editors
> Done. L10N will be done afterwards.
>
>
[snip]

> Property changes on: 
> platform/web/trunk/standard/src/main/webapp/resources/icons/datamodel/rights.png
> ___________________________________________________________________
> Name: svn:mime-type
>    + image/png
>
>
This image is not very comprehensive. I had to look twice before getting 
it is key ring.

[snip]

> +XWiki.XPropertyOrdering = Class.create({

Shouldn't this be namespaced under the XWiki namespace ?

For example "XWiki.dataeditor.XPropertyOrdering"

If we do not make submodules in the XWiki objects, we'll make the same 
mistake we did with the window object, and we will end up with a large 
collection of properties and functions that have nothing to do one with 
each other.
I'd rather keep the XWiki object for both holding modules and 
functions/classes/properties that have sense in all XWiki pages, not 
just in one module.

Jerome.

> _______________________________________________
> notifications mailing list
> [email protected]
> http://lists.xwiki.org/mailman/listinfo/notifications

_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to