millennium-dev team mailing list archive

[Jakub Sękowski <sequba@xxxxxxxxx>]

I was thinking about this issue for a while and my idea is to make
"_margin" a private member of IDrawable (similar to
"_background_color"). But this will cause some problems.

A lot of methods, such as OnClicked(...) and GetMinimal...(...), have to
be able to determine widget's margin, so I suggest, to use similar trick
to the one, Rafał did with Draw(...) - make those functions non-virtual
public methods of IDrawable and specify something like
OnClickedNoMargin(...) and GetMinimal...NoMargin(...) (analogous to
Redraw(...)), which will ignore widget's margin completely.

However, I'm not sure if this is the right way to do it. Otherwise, I
can just specify public "getter" method GetMargin(), which, obviously,
would be called by OnClicked(...) and GetMinimal...(...), but approach
is completely different to the "_background_color"-one.

What do you think about it?

-- 
You received this bug notification because you are a member of
Millennium Developers, which is subscribed to Millennium Duel.
https://bugs.launchpad.net/bugs/1204286

Title:
  Margin should be all Drawables' property.

Status in Millennium Duel - a GUI frontend for Millennium:
  Triaged

Bug description:
  I suggest to add member "margin" and method "SetMargin" to Drawable
  class definition. Why only Boxes may have it?

To manage notifications about this bug go to:
https://bugs.launchpad.net/millenniumduel/+bug/1204286/+subscriptions