search

Driftwood2D / Driftwood

Driftwood 2D Tiling Game Engine and Development Suite
http://tileengine.org/
MIT License
24 stars 1 forks source link

Clean up duplicate code in WidgetManager #81

Closed seisatsu closed 7 years ago

seisatsu commented 7 years ago

The container and text functions have a ton of duplicate code. It should be merged.

pmer commented 7 years ago

I think they would make good candidates for two new classes: ContainerWidget and TextWidget.

The base Widget class they derive from can contain the shared functionality.

What do you think?

seisatsu commented 7 years ago

Could work. I'll take a look at how I might want to do that.

seisatsu commented 7 years ago

As of abea37842bc16cfdfb14168966b99604757ea48a we've probably done most of what we can reasonably do by making the ContainerWidget and TextWidget subclasses.