QML style guide

[Aaron J. Seigo]

On Tuesday, October 30, 2012 15:01:04 Dmitry Ashkadov wrote:
> 30.10.2012 14:35, Aaron J. Seigo пишет:
> >>>> 5. Vim:
> >>>>> The {{{ and }}} are just for the sakes of vim's automatic folding.
> >>>> 
> >>>> Do we all use vim? I think no. So, we must never force developers use
> >>>> vim.
> >>> 
> >>> i mostly agree :) i don't think this forces anyone to use vim, but i
> >>> would
> >>> like to see a solution that works with katepart before adding it to the
> >>> style guide.
> >> 
> >> Do you want force developers use kate?
> > 
> > of course not :)
> > 
> > but i'd like to make the lives of people who use kate, vim, etc. easier if
> > possible. if we have to chose between "no one benefits from code folding"
> > or "some people get to benefit", we should pick "some people" every
> > single time. "some" is better than "none".
> > 
> > this does not prevent you from using any editor you wish -> you might not
> > get the cute little bonus of code folding, but otherwise it's the same as
> > it is right now already.
> 
> An editor should handle braces and provide folding functionality. I
> don't think, that forcing all developer always put "{{{"  and "}}}" in
> addition to braces { }, is good idea.

the suggestion was to only do this for the different sections such as property 
definitions. example:

Item {
    id: foo

    /* property definitions {{{ */
    property bar: true
    inheritedProperty: 123
    /* }}} */
        
    /* JS functions {{{ */
    function myFunc() {
    }

    function myOtherFunc() {
    }
    /* }}} */
}

the idea is that lets you fold down allthe js functions or all the property 
definitions.

> Let it be first (geometry goes first):

cool, one more vote for geom first :) so noted ...

-- 
Aaron J. Seigo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121030/774a826b/attachment.sig>