Open GoogleCodeExporter opened 9 years ago
Original comment by vendett...@gmail.com
on 19 Apr 2012 at 1:48
This issue was updated by revision r346.
Added variable isActive representing whether the button can be clicked or not.
Added getter for position.
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:22
This issue was updated by revision r347.
Added isActive to isClicked() method.
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:28
This issue was updated by revision r348.
Uploaded LevelSelect class
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:41
This issue was updated by revision r349.
Uploaded resources for LevelSelect class
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:45
This issue was updated by revision r350.
Fixed small error in using statements.
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:55
This issue was updated by revision r351.
Fixed another tiny error in using statements.
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 6:59
This issue was updated by revision r352.
initialized isActive
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 7:03
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 7:14
This issue was updated by revision r361.
reverted Button back to original
and added array isActive to levelSelect
Original comment by vendett...@gmail.com
on 23 Apr 2012 at 8:27
This issue was updated by revision r398.
Added file to the solution as it wasnt in the solution
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 11:11
This issue was updated by revision r400.
added resources and solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 11:52
This issue was updated by revision r401.
updated solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 11:58
This issue was updated by revision r402.
changed loading textures
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 12:07
This issue was updated by revision r403.
updated solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 12:13
This issue was updated by revision r404.
updated class LevelSelect.cs
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 12:33
This issue was updated by revision r406.
removed DrawableGameComponent inheritance
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 12:41
This issue was updated by revision r410.
updated solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 12:49
This issue was updated by revision r411.
updated solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 1:23
This issue was updated by revision r416.
updated LevelSelect.cs
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 1:42
This issue was updated by revision r417.
updated solution file
Original comment by vendett...@gmail.com
on 24 Apr 2012 at 1:44
This issue was updated by revision r531.
Fixed some bugs
Original comment by vendett...@gmail.com
on 25 Apr 2012 at 9:21
This issue was updated by revision r580.
Modified to meet the conventions of Button
Original comment by vendett...@gmail.com
on 26 Apr 2012 at 1:07
This issue was updated by revision r592.
added some features
Original comment by vendett...@gmail.com
on 26 Apr 2012 at 6:53
-> Implementation review for LevelSelect class
1 - Constructor
* ContentManager parameter should be passed instead of Game game
2 - Initialize
* the number of buttons should not be restricted to 3 and their inializations
should be done in a loop
* a textureStrips array should be used and its initialization should be done
using a loop
3 - Update
* the buttons isClicked checks should be done in a loop
4 - naming conventions are fine
Original comment by bishoyba...@gmail.com
on 26 Apr 2012 at 10:08
2- Yes, they should be restricted to 3, you obviously dont know what these
buttons represent. These buttons are just the places of the actual level
buttons which arent defined. Plus.. why do i need a loop if each is initialized
differently ???
3- Each button has a different action?? loop ??
Original comment by vendett...@gmail.com
on 26 Apr 2012 at 10:21
This issue was updated by revision r648.
fixed bugs
Original comment by vendett...@gmail.com
on 27 Apr 2012 at 12:45
This issue was updated by revision r649.
added resources
Original comment by vendett...@gmail.com
on 27 Apr 2012 at 12:47
This issue was updated by revision r650.
added resources
Original comment by vendett...@gmail.com
on 27 Apr 2012 at 12:50
Original comment by ahmedheg...@gmail.com
on 27 Apr 2012 at 5:35
Original comment by ahmedheg...@gmail.com
on 27 Apr 2012 at 6:33
This issue was updated by revision r1426.
Fixed.
Original comment by vendett...@gmail.com
on 23 May 2012 at 1:35
This issue was updated by revision r1427.
Fixed.
Original comment by vendett...@gmail.com
on 23 May 2012 at 1:36
when hovering on a level it shows 3 blue boxes over each others
Original comment by khaled.osmaan
on 23 May 2012 at 5:30
This issue was updated by revision r1488.
Fixed.
Original comment by vendett...@gmail.com
on 23 May 2012 at 5:53
small errors since everything is hard coded. lol
should work now
Original comment by vendett...@gmail.com
on 23 May 2012 at 6:00
This issue was updated by revision r1498.
Added levels
Original comment by vendett...@gmail.com
on 23 May 2012 at 7:10
class LevelSelect:
*delete unused using statements.
*all instance variable should be private.
*The arguments should be a full name, not just some characters.
*Initialize variables in the constructor. Load the pictures in LoadContent()
*Update(): switch(level): make a seperate method that gets the selected level.
and remove this part. It makes it more usable.
*Draw(): make calculations and rectangles in variables and then use it.
Original comment by ahmedheg...@gmail.com
on 24 May 2012 at 10:39
This issue was updated by revision r1682.
Fixed. Finally.
Original comment by vendett...@gmail.com
on 24 May 2012 at 2:45
its possible to hover over and select more than one screen, I'm not sure its
your fault or hegazy's
Original comment by khaled.osmaan
on 24 May 2012 at 8:39
Original issue reported on code.google.com by
vendett...@gmail.com
on 13 Apr 2012 at 6:24