- User Since
- Apr 17 2017, 9:58 PM (129 w, 6 d)
Tue, Sep 17
It looks good to me.
All calls to this function with unused arguments have been removed.
Jun 19 2019
Jun 5 2019
Just create an object literal.
Why not just use Singleton pattern instead using prototypes if only one instance?
Jun 1 2019
You can remove descriptions if they are not really descriptive and used for the moment .
Maybe change the name of GetTypes() -> GetCodes() to reflect what it really returns and to be consistent with resources.
You can remove some return true.
May 31 2019
For the translation, you need to focus only in the GUI part.
Look specially to common/tooltips.js
May 25 2019
resources parameter in GetTemplateDataHelper() became unused in the function.
May 20 2019
Test is better than words.
The problem with the current code is that at no time we recover the real size of the backgrounds.
Depending on the window size the tiling backgrounds are more or less fully displayed.
For the rounding I thought about ceil rounding timeRemaining in GetQueue().
But I don't know if it's the best thing to do or the best place.
You forgot to put the full context to the diff.
var -> let
May 19 2019
- Moves MAX_QUEUE_SIZE into the prototype.
- Integrates the lateness parameter from Timers.
- Also fix errors forgotten according to the comments.
May 18 2019
The timers can have lateness, should not we include this?
May 12 2019
Tested and its works as expected in relation to the ticket.
Move the decrementation entity limit only when the entity has been successfully spawned and before the ownership attribution seems correct.
May 6 2019
You are right but it's not easier and usual to use length form to iterate in an array?
this.mirages is an array.
Apr 27 2019
Better to separate this into several differentials and with real things to cleanup.
Refactor RemoveBatch function.
Then correct the length check entityCache.
Apr 25 2019
So it was intended. :d
Apr 24 2019
It should be determined if we want certain elements of the background can't be stretched.
But it can also cause this sprite could be too small compared to other elements for wide screens.
Use a hack instead of JS modifications.
The JS changes were almost the same as the code for tilling.
Apr 22 2019
The Icons of units concerned are no longer displayed.
You forgot to include the renaming pictures with the new names.
Eg: hele_mechanical_siege_lithobolos_packed.png -> hele_siege_lithobolos_packed.png
Mar 8 2019
You could use "Update Diff" in right panel or update via arch.
Feb 16 2019
He means the context lines (lines before and after the codes that are modified).
For each patch, you must add the full context.
To add these lines, refer here.
Feb 7 2019
One ticket about this. #3800
What do you mean by the sprite's attribute?
Feb 6 2019
Disables the notification by default.
Change user to client in option's description.
Feb 5 2019
You are right. I will change that.
Jan 31 2019
Upload the sound.
New update with a new dedicated Bell tree sound.
Thank's to @Samulis for the creation.
Jan 25 2019
Maybe try rather adapts the code with the fonction enunciate by smiley.
Jan 24 2019
Maybe a better balcony scale.
And remove the effect.
- Rephrase in options.
- Changes according to Imarok comments. (don't hide all messages of currents players)
I didn't really notice the clouds move too fast but maybe its just me.
Jan 18 2019
Oct 18 2018
Oct 17 2018
Apr 22 2018
Dec 20 2017
Nov 17 2017
The number of games can be split above the GamePanel imo.
I think it can be useful when a multiplayer game starts after the countdown.
But this is probably the only way this function can appear.
Idk if the new globals.js file is needed, there isn't another file where this function can be placed?
Oct 25 2017
Oct 13 2017
This countdown can be useful also if added with a countdown sound.
2 differents sounds:
- during the countdown
- and when the game start really.
If the idea is to find the game where the player was playing, i think it would be better to display a pop-up to reconnect to the game when the player is in the lobby.
Oct 2 2017
Sep 28 2017
Sep 27 2017
Sep 25 2017
Nice add and it will be useful for future notifications (e.g. Friend Connection in lobby or others stuffs like that).
I noticed you remove the splitRating part.
Sep 23 2017
Sep 22 2017
Forgot the capital letter in g_SoundNotifications
Sep 21 2017
Sep 20 2017
Aug 21 2017
Jun 13 2017
tutorials.po files are not generated.
Translation on transifex aren't taken into account.
Jun 9 2017
Jun 7 2017
Jun 6 2017
Filenames seem to look more like:
"animal_species" than "species_animal".
It will be "wolf_arctic" for all the filenames.
Jun 2 2017
May 24 2017
I don't like the "if the map places it" sentence.
May 22 2017
Full context this time.
reupload with context
Remove the cleanup file part.