Page MenuHomeWildfire Games

Vantha ( )
User

Projects

User Details

User Since
Mar 30 2024, 10:25 PM (7 w, 4 d)

Recent Activity

Tue, May 21

Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Tue, May 21, 7:10 PM
Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Tue, May 21, 6:02 PM
Vantha added inline comments to D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Tue, May 21, 6:00 PM
Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.

removed the extra parameter textColorSelected from COList::DrawList()

Tue, May 21, 5:56 PM
Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.

This should prevent the compiler warnings.

Tue, May 21, 2:17 PM

Mon, May 20

Vantha added inline comments to D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Mon, May 20, 9:36 PM
Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.

updated the copyright years

Mon, May 20, 9:36 PM

Sun, May 19

Vantha added a comment to D5269: Finish implementing property "textcolor_selected" for list GUI objects.

Ok. I removed the whitespace.

Sun, May 19, 11:49 AM
Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Sun, May 19, 10:56 AM
Vantha added a comment to D5269: Finish implementing property "textcolor_selected" for list GUI objects.

Thanks.
Are the problematic whitespaces still there? I couldn't find them in the files, but the inline comment is gone now.

Sun, May 19, 12:14 AM

Sat, May 18

Vantha updated the diff for D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Sat, May 18, 11:59 PM
Vantha requested review of D5269: Finish implementing property "textcolor_selected" for list GUI objects.
Sat, May 18, 12:01 AM

Thu, May 9

Vantha added a comment to D5256: Remove Roman numerals from bot names.

Nice. Is there anything else you want me to change?

Thu, May 9, 4:17 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Thu, May 9, 4:16 PM
Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Thu, May 9, 11:14 AM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Thu, May 9, 11:10 AM

Wed, May 8

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Wed, May 8, 11:14 PM
Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Wed, May 8, 2:24 PM

Tue, May 7

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Tue, May 7, 6:52 PM
Vantha added a comment to D5256: Remove Roman numerals from bot names.
Tue, May 7, 6:48 PM
Vantha added a comment to D5256: Remove Roman numerals from bot names.

elexis said that this revision should not be marked as closing ticket #3307 because it doesn't resolve all of that ticket. So, remove the reference to ticket #3307 in the ticket field. Then add a comment to the description stating that this resolves part of ticket #3307.

Tue, May 7, 1:49 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Tue, May 7, 1:45 PM

Sun, Apr 28

Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Sun, Apr 28, 8:14 PM

Fri, Apr 26

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Fri, Apr 26, 9:16 AM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Fri, Apr 26, 9:16 AM

Wed, Apr 24

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Wed, Apr 24, 9:14 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.

I addressed most of the suggestions.

Wed, Apr 24, 9:13 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.

Ok, updated the diff. Is this how you wanted the comments?

Wed, Apr 24, 3:29 PM

Tue, Apr 23

Vantha updated the diff for D5256: Remove Roman numerals from bot names.

Like this?

Tue, Apr 23, 8:30 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.

I added/updated the comments. Do you also want a comment before line 95? This.CountLabel is already marked for translation at this point (and has a comment explaining what it is).

Tue, Apr 23, 12:20 PM

Apr 21 2024

Vantha updated the diff for D5256: Remove Roman numerals from bot names.

Ok, nice. Made the change and added the trac ticket.

Apr 21 2024, 12:11 AM

Apr 20 2024

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Apr 20 2024, 12:02 AM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.

Ok, I made the change.

Apr 20 2024, 12:01 AM

Apr 19 2024

Vantha added a comment to D5256: Remove Roman numerals from bot names.

OK, and what's preferred?

Apr 19 2024, 5:34 PM

Apr 18 2024

Vantha added a comment to D5256: Remove Roman numerals from bot names.

Before this diff the "forat translation" is allways done. That's almost certainly an issue.

Not sure what you mean by this, though. What should I change?

Apr 18 2024, 6:51 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Apr 18 2024, 6:49 PM
Vantha added a comment to D5256: Remove Roman numerals from bot names.

I don't like it when data is mutated. (With the +=)
If one wants to know what's in the variable one has to reason about every point where it get's mutated.
If it's const one only has to reason abot the initialization.
Ideally there should be a chosenName and a translatedChosenName:

const translatedChosenName = this.settings.isNetworked ? chosenName : translate(chosenName);

elexis just wrote me again :)
User visible strings shouldn't be concatinated by + (or +=).

Apr 18 2024, 3:01 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Apr 18 2024, 2:38 PM

Apr 17 2024

Vantha added inline comments to D5256: Remove Roman numerals from bot names.
Apr 17 2024, 8:22 PM
Vantha added a comment to D5256: Remove Roman numerals from bot names.

I never saw Template Literals in this codepase. I don't object it. :)

Apr 17 2024, 8:19 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.
Apr 17 2024, 8:03 PM

Apr 8 2024

Vantha updated the diff for D5256: Remove Roman numerals from bot names.

The Mauryan emperors were sorted chronologically. I put them in alphabetical order too.

Apr 8 2024, 6:22 PM
Vantha added a comment to D5256: Remove Roman numerals from bot names.

When the AINames are empty or don't exist, the game already throws errors before the step of choosing the names.

Apr 8 2024, 1:37 PM
Vantha updated the diff for D5256: Remove Roman numerals from bot names.

changing three let to const

Apr 8 2024, 1:30 PM

Apr 7 2024

Vantha requested review of D5256: Remove Roman numerals from bot names.
Apr 7 2024, 3:29 PM