Method formatting no longer works properly

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Method formatting no longer works properly

Intrader Intrader
After loading latests configuration.
The comment format(r) is strangely converting
<prior to format>
renderContentOn: html
    html anchor
        callback: [ self addContact ];
        with: 'Add contact'.
    "bad dessign to refer to MyContact class directly; better to add model
instance variable"
    html unorderedList: [
        MyContact contacts do: [ :contact |
        html listItem: [ self renderContact: contact on: html ] ] ]
</prior to format>
<after format>
renderContentOn: html
        (html anchor)
                callback: [ self addContact ];
                with: 'Add contact'. "bad design to refer to MyContact class
 directly; better to add model instance variable"
html unorderedList: [ MyContact contacts do: [ :contact | html listItem:
 [ self renderContact: contact on: html ] ] ]
</after format>
I don't understand:
1. the parenthesis use around (html anchor).
2. the comment position
3. the one one starting with html unorderedList is not formatted at all.

My current version is
Pharo 1.1.1
Latest update #11414

Thanks for any help.


 


_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Method formatting no longer works properly

Philippe Marschall
2011/2/15 Fritz Schenk <[hidden email]>:

> After loading latests configuration.
> The comment format(r) is strangely converting
> <prior to format>
> renderContentOn: html
>    html anchor
>        callback: [ self addContact ];
>        with: 'Add contact'.
>    "bad dessign to refer to MyContact class directly; better to add model
> instance variable"
>    html unorderedList: [
>        MyContact contacts do: [ :contact |
>        html listItem: [ self renderContact: contact on: html ] ] ]
> </prior to format>
> <after format>
> renderContentOn: html
>        (html anchor)
>                callback: [ self addContact ];
>                with: 'Add contact'.    "bad design to refer to MyContact class
>  directly; better to add model instance variable"
> html unorderedList: [ MyContact contacts do: [ :contact | html listItem:
>  [ self renderContact: contact on: html ] ] ]
> </after format>
> I don't understand:
> 1. the parenthesis use around (html anchor).
> 2. the comment position
> 3. the one one starting with html unorderedList is not formatted at all.
>
> My current version is
> Pharo 1.1.1
> Latest update #11414

This should be reported against Pharo:
http://pharo-project.org/community

Cheers
Philippe
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Method formatting no longer works properly

Intrader Intrader
Philippe, thanks
I am following up in http://webchat.freenode.net/?channels=pharo-project as intrader.

_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Method formatting no longer works properly

Intrader Intrader
In reply to this post by Philippe Marschall
Philippe, I have installed Pharo 1.2, and still have the problem. The format(r)
is just not working properly.


_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: Method formatting no longer works properly

Intrader Intrader
In reply to this post by Philippe Marschall
Philippe, I have commented an existing bug report with the similar problem:
http://code.google.com/p/pharo/issues/detail?id=2820

_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside