S14e ("Saorduc Swamps") minor nits

nemaara / A_New_Order

add-on campaign for The Battle for Wesnoth featuring the Akladians; originally by szopen

https://forums.wesnoth.org/viewtopic.php?f=8&t=6486

GNU General Public License v2.0

10 stars 7 forks source link

S14e ("Saorduc Swamps") minor nits #33

Open cooljeanius opened 2 years ago

cooljeanius commented 2 years ago

[ ] Maybe the lizards should be able to join you? I dunno, it's just something Marximilian suggested... (also be sure to test dialogue in that area, too)
[x] Make a macro to reduce repetition of the moveto events
[ ] Ensure modifications to saurians to represent their sluggishness in the cold actually work properly (and consider updating them as necessary)
[ ] unused macro cleanup; see issue #57 (for example, the PROPOSE_PEACE_LIZARD macro)

cooljeanius commented 2 years ago

I'm thinking of tweaking the map a bit in this one, to make more of the terrain snowy, which might mean needing to tweak the terrain filter for their "cold-bloodedness makes them sluggish" event... I dunno, though...

cooljeanius commented 2 years ago

I'm thinking of tweaking the map a bit in this one, to make more of the terrain snowy, which might mean needing to tweak the terrain filter for their "cold-bloodedness makes them sluggish" event... I dunno, though...

so yeah the map tweaks went in as part of #75; I didn't actually adjust the filters at all, though, although I did at least add a comment noting it in #76...

cooljeanius commented 1 year ago

I'm thinking of tweaking the map a bit in this one, to make more of the terrain snowy, which might mean needing to tweak the terrain filter for their "cold-bloodedness makes them sluggish" event... I dunno, though...

so yeah the map tweaks went in as part of #75; I didn't actually adjust the filters at all, though, although I did at least add a comment noting it in #76...

Further map tweaks may be in order if wesnoth/wesnoth#6752 is ever fixed

cooljeanius commented 1 year ago

additional checkbox added to OP; see issue #57 (re: the PROPOSE_PEACE_LIZARD macro)

cooljeanius commented 9 months ago

I'll be able to check off the 2nd checkbox after creating and merging a PR that includes cooljeanius/A_New_Order@2f58117

cooljeanius commented 4 months ago

I'll be able to check off the 2nd checkbox after creating and merging a PR that includes cooljeanius/A_New_Order@2f58117

Update: done in #163