Issue 839 in moose-technology: ROCenteredLabel

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

Issue 839 in moose-technology: ROCenteredLabel

moose-technology
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

I required text to be centered within a circle, so I created  
ROCenteredLabel. I think it would be generally useful. Would you consider  
adding it to Roassal?


Attachments:
        ROCenteredLabel-drawOnfor.st  1.0 KB

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 839 in moose-technology: ROCenteredLabel

moose-technology

Comment #1 on issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

I just noticed the fileout didn't include the class definition.  Probably  
obvious but for completeness...

ROLabel subclass: #ROCenteredLabel
        instanceVariableNames: ''
        classVariableNames: ''
        poolDictionaries: ''
        category: 'Roassal-Core'

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 839 in moose-technology: ROCenteredLabel

moose-technology

Comment #2 on issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

whoops - remove redundant 'oldOffset' tempVar

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 839 in moose-technology: ROCenteredLabel

moose-technology
Updates:
        Status: Started
        Labels: -Priority-Medium Priority-High Component-Roassal

Comment #3 on issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

Thanks Ben!
This is an excellent contribution!

I've improved the label implementation. ROLabel is now a subclass of  
ROAbstractLabel. ROCenteredLabel is a subclass of ROAbstractLabel instead  
of ROLabel.
I also refactored the tests. The Mondrian builder offers #withCenteredText  
and #withCenteredText:
An example in Mondrian/Basic has been added.

-=-=-=-=-=-=-=-=-=-=-=-=
        view shape rectangle; withCenteredText;  width: 180.
        view node: 'centered text'.

        view shape rectangle; withText;  width: 180.
        view node: 'left text'.
-=-=-=-=-=-=-=-=-=-=-=-=

Excellent piece of work!


_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 839 in moose-technology: ROCenteredLabel

moose-technology
Updates:
        Status: Fixed

Comment #4 on issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

In Roassal 1.125

_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|

Re: Issue 839 in moose-technology: ROCenteredLabel

moose-technology
Updates:
        Labels: Milestone-4.7

Comment #5 on issue 839 by [hidden email]: ROCenteredLabel
http://code.google.com/p/moose-technology/issues/detail?id=839

(No comment was entered for this change.)

--
You received this message because this project is configured to send all  
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.iam.unibe.ch/mailman/listinfo/moose-dev