Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 184982

Summary:

[Contribution] RoundedLineBorder

Product:

[Tools] GEF

Reporter:

Adam Cabler <acabler>

Component:

GEF-Legacy Draw2d

Assignee:

gef-inbox <gef-inbox>

Status:

RESOLVED WORKSFORME

QA Contact:

Severity:

enhancement

Priority:

CC:

nyssen

Version:

3.3

Target Milestone:

---

Hardware:

OS:

Windows NT

Whiteboard:

Attachments:

Description	Flags
code contribution	none

Description Adam Cabler

2007-05-01 17:01:27 EDT

I needed a quick solution to have a border for rounded rectangles.  This works great for me and is really simple.  I searched to see if there is something already contributed, but since there isn't...

Comment 1 Adam Cabler

2007-05-01 17:03:24 EDT

Created attachment 65533 [details]
code contribution

Comment 2 Randy Hudson

2007-05-02 16:26:43 EDT

This doesn't look so hot if the figure is setOpaque(true), since that fills a normal rectangle. See AbstractBackground, which was just added in 3.3 to allow a border like this one to also fill in the rounded rectangle.

Comment 3 Adam Cabler

2007-05-02 16:42:18 EDT

That must be since M5.  I'm not able to upgrade until a few things are fixed, so I don't have it to look at.  Are you saying that AbstractBackground can be extended to better serve this purpose?  Btw - I'm just looking for a border, not a fill.  Or are you saying to use AbstractBackground to fill the rrect except for the border to serve the same purpose?



(In reply to comment #2)
> This doesn't look so hot if the figure is setOpaque(true), since that fills a
> normal rectangle. See AbstractBackground, which was just added in 3.3 to allow
> a border like this one to also fill in the rounded rectangle.
>

Comment 4 Randy Hudson

2007-05-02 17:36:22 EDT

You've essentially extracted a portion of RoundedRectangle into a border. Unless you extract all of it into the Background support, there's not really much there to warrant adding it to an already over-populated package (draw2d).

Comment 5 Alexander Nyßen

2011-01-08 07:50:39 EST

Resolving due to last comment and because the bug has been inactive since 2007.