If you look closely, you'll see there is a break in the hem of the " skirt", showing that they're legs to loose culottes. Also, I believe it was Marston's wife that said they should be culottes ( who needs a skirt flying up over her head? Lol)
I'm ok with each Earth's Diana having a look refective of that era/ continuity.( So we get all the looks on each appropriate Diana)
For me, if we're going with the current version of her costume, then here are my requests:
(1)"battle skirt" should be short, like in the movie. Once it gets long and bulky, it looks ridiculous.
(2) her emblem should not wrap all the way around her torso, and should be framed along the top by the red of the bodice
(3) no cape, no sword, no shield unless under special circumstances ( ceremonial events, or major battle against cosmic forces...though let's remember, Bronze Age Diana fought in COIE without armor )
Oh, and all Amazons of Themyscira should be wearing their religious icons....the bracelets .