← Back to team overview

millennium-dev team mailing list archive

Re: [Merge] lp:~rafalcieslak256/millenniumduel/containers into lp:millenniumduel

 

Review: Needs Fixing

I love the code you wrote. It's elegant and very powerful, and is going to save us a lot of effort. 

However, I discovered a minor bug. When examined carefully, it turns out, that buttons' click sensivity areas are a bit offset upwards with respect to their drawings. 
I think I can tell which part of code needs a correction. In functions HBox::on_clicked and VBox::on_clicked, the 'sum' variable should be initialized with:

int sum = margin;

instead of actual:

int sum = 0; 


Very nice job!
-- 
https://code.launchpad.net/~rafalcieslak256/millenniumduel/containers/+merge/176052
Your team Millennium Developers is subscribed to branch lp:millenniumduel.


References