Closed GoogleCodeExporter closed 9 years ago
Original comment by q3a...@gmail.com
on 11 May 2012 at 8:52
Original comment by q3a...@gmail.com
on 11 May 2012 at 8:53
Original comment by q3a...@gmail.com
on 12 May 2012 at 9:40
Original comment by q3a...@gmail.com
on 14 May 2012 at 7:33
This issue was updated by revision r869.
Added Fading Screen class that will be used to create the fading logos for the
intro screen
Original comment by q3a...@gmail.com
on 14 May 2012 at 7:34
This issue was updated by revision r871.
Adding resources that will be used inside the screens
Original comment by q3a...@gmail.com
on 14 May 2012 at 7:38
This issue was updated by revision r873.
Changed the formating for the class
Original comment by q3a...@gmail.com
on 14 May 2012 at 7:41
This issue was updated by revision r874.
Committing the project to have the changes in the solution updated
Original comment by q3a...@gmail.com
on 14 May 2012 at 7:42
This issue was updated by revision r898.
Adding resources that will be used inside the screens
Original comment by q3a...@gmail.com
on 14 May 2012 at 11:03
This issue was updated by revision r912.
Updated the fading values
Updated the constructor
Added new fields
Changed the background color to white
Original comment by q3a...@gmail.com
on 15 May 2012 at 12:07
This issue was updated by revision r914.
Added Screens for the intro screen
Original comment by q3a...@gmail.com
on 15 May 2012 at 12:16
This issue was updated by revision r915.
Added resources that will be used by intro screens
Original comment by q3a...@gmail.com
on 15 May 2012 at 12:17
This issue was updated by revision r916.
Added everything related to this story to the project solution
Fixed some errors
Original comment by q3a...@gmail.com
on 15 May 2012 at 12:30
This issue was updated by revision r922.
Added documentation
Original comment by q3a...@gmail.com
on 15 May 2012 at 2:36
Original comment by q3a...@gmail.com
on 15 May 2012 at 2:36
This issue was updated by revision r923.
Added documentation to class GUCScreen
Original comment by q3a...@gmail.com
on 15 May 2012 at 2:38
This issue was updated by revision r924.
Added documentation to class ITworxScreen
Renamed class from ITWroxScreen to ITworxScreen
Original comment by q3a...@gmail.com
on 15 May 2012 at 2:42
This issue was updated by revision r925.
Added documentation to class TeraSoftScreen
Original comment by q3a...@gmail.com
on 15 May 2012 at 2:46
This issue was updated by revision r992.
Modified the way fading screen works to make it more customizable and scalable
Added Documentation to new classes
Removed GUCScreen class
GUCLogo is now in the same screen with terasoft
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:42
This issue was updated by revision r993.
Updating solution with the delete of GUCScreen
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:43
This issue was updated by revision r994.
Deleted GUCScreen.cs from the files
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:44
This issue was updated by revision r995.
Updating solution with the delete of GUCScreen
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:45
This issue was updated by revision r996.
Updating solution with the delete of GUCScreen
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:45
This issue was updated by revision r997.
Updated documentation after the latest changes
Original comment by q3a...@gmail.com
on 17 May 2012 at 11:52
This issue was updated by revision r1002.
Deleted ITWorxScreen.cs wrong file
Original comment by q3a...@gmail.com
on 17 May 2012 at 12:36
This issue was updated by revision r1005.
Updated Solution for the new renaming
Original comment by q3a...@gmail.com
on 17 May 2012 at 12:40
https://cacoo.com/diagrams/NVODQy9B9HfXrvEZ
Original comment by q3a...@gmail.com
on 17 May 2012 at 1:17
https://cacoo.com/diagrams/bCih4M7NGtzgKH3H
Original comment by q3a...@gmail.com
on 17 May 2012 at 1:18
This issue was updated by revision r1107.
Replaced old Update method with the new one as old one will be deprecated
Original comment by q3a...@gmail.com
on 18 May 2012 at 1:15
Original comment by q3a...@gmail.com
on 20 May 2012 at 11:35
Original comment by q3a...@gmail.com
on 21 May 2012 at 11:28
This issue was updated by revision r1313.
Original comment by q3a...@gmail.com
on 22 May 2012 at 8:27
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
This issue was updated by revision r1394.
Disabled the avatar in my screen.
Original comment by q3a...@gmail.com
on 23 May 2012 at 10:21
The class Fading Screen lacks author tags.
/// <param name="path">path of the image that will be added to the middle of
the screen.</param>
/// <param name="logoScale">scaling ratio of the image.</param>
/// <param name="rotation">rotation for the image.</param>
Capitalize all three, and I don't understand path. Please make it more clear.
Also what is rotation mean exactly? Is it degrees or radians or what? I think
you should specify what it is.
represents the time of the game.
Capitalize.
specifies wether the screen is covered.
capitalize and spelling "wether"
represents the time of the game.
Capitalize (Same mistake, in the same exact sentence in two different methods)
Original comment by ams.sa...@gmail.com
on 23 May 2012 at 11:22
Class ITworxScreen lacks author tags.
/// <summary>
/// Updates this screen
/// </summary>
/// <param name="gameTime">represents the time of the game.</param>
/// <param name="covered">specifies wether the screen is covered.</param>
Add a '.'
Capitalize
Capitalize and spelling mistake "wether"
represents the time of the game.
Capitalize (again)
Original comment by ams.sa...@gmail.com
on 23 May 2012 at 11:25
All classes in this story (including the ones I didn't review previously) seem
to have the same problem exactly (seems you copy pasted the documentation)
No author tags at all
"Updates this screen" lacks a '.'
"represents the time of the game." needs to be capitalized (twice)
"specifies wether the screen is covered." Capitalized and spelling mistake in
"wether"
Please check them all and fix these mistakes.
Original comment by ams.sa...@gmail.com
on 23 May 2012 at 11:29
https://cacoo.com/diagrams/NVODQy9B9HfXrvEZ
https://cacoo.com/diagrams/bCih4M7NGtzgKH3H
Original comment by q3a...@gmail.com
on 23 May 2012 at 6:53
This issue was updated by revision r1543.
Made the screens scalable
Changed the scale variable semantically
Original comment by q3a...@gmail.com
on 23 May 2012 at 10:55
This issue was updated by revision r1645.
Used new Update method, because the other one is deprecated
Original comment by q3a...@gmail.com
on 24 May 2012 at 11:08
This issue was updated by revision r1652.
Removed an unused FadingScreen constructor
Fixed FadingScreen documentation
Original comment by q3a...@gmail.com
on 24 May 2012 at 11:24
This issue was updated by revision r1653.
Fixed documentation of ITWorxScreen and TeraSoftScreen
Original comment by q3a...@gmail.com
on 24 May 2012 at 11:28
This issue was updated by revision r1654.
Added author tag to all the classes
Original comment by q3a...@gmail.com
on 24 May 2012 at 11:30
This issue was updated by revision r1666.
Fixed lines that were longer than 120 chars
Original comment by q3a...@gmail.com
on 24 May 2012 at 12:23
Original issue reported on code.google.com by
q3a...@gmail.com
on 11 May 2012 at 8:10