[ECC] a few text fixes

Mage.Sets/src/mage/cards/e/ElementalSpectacle.java

@@ -1,19 +1,19 @@
 package mage.cards.e;
 
-import java.util.UUID;
-
 import mage.abilities.dynamicvalue.common.ColorsAmongControlledPermanentsCount;
 import mage.abilities.dynamicvalue.common.CreaturesYouControlCount;
 import mage.abilities.effects.common.CreateTokenEffect;
 import mage.abilities.effects.common.GainLifeEffect;
+import mage.abilities.hint.common.CreaturesYouControlHint;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
 import mage.constants.AbilityWord;
 import mage.constants.CardType;
 import mage.game.permanent.token.OmnathElementalToken;
 
+import java.util.UUID;
+
 /**
- *
  * @author muz
  */
 public final class ElementalSpectacle extends CardImpl {
@@ -23,13 +23,14 @@ public final class ElementalSpectacle extends CardImpl {
 
         // Vivid -- Create a number of 5/5 red and green Elemental creature tokens equal to the number of colors among permanents you control. Then you gain life equal to the number of creatures you control.
         this.getSpellAbility().addEffect(new CreateTokenEffect(
-            new OmnathElementalToken(),
-            ColorsAmongControlledPermanentsCount.ALL_PERMANENTS
-        ));
+                new OmnathElementalToken(), ColorsAmongControlledPermanentsCount.ALL_PERMANENTS
+        ).setText("create a number of 5/5 red and green Elemental creature tokens " +
+                "equal to the number of colors among permanents you control"));
         this.getSpellAbility().addEffect(new GainLifeEffect(CreaturesYouControlCount.PLURAL)
-            .setText(", then you gain life equal to the number of creatures you control"));
+                .setText("Then you gain life equal to the number of creatures you control"));
         this.getSpellAbility().setAbilityWord(AbilityWord.VIVID);
         this.getSpellAbility().addHint(ColorsAmongControlledPermanentsCount.ALL_PERMANENTS.getHint());
+        this.getSpellAbility().addHint(CreaturesYouControlHint.instance);
     }
 
     private ElementalSpectacle(final ElementalSpectacle card) {

Mage.Sets/src/mage/cards/g/GlissaSunslayer.java

@@ -12,7 +12,9 @@ import mage.abilities.keyword.DeathtouchAbility;
 import mage.abilities.keyword.FirstStrikeAbility;
 import mage.cards.CardImpl;
 import mage.cards.CardSetInfo;
-import mage.constants.*;
+import mage.constants.CardType;
+import mage.constants.SubType;
+import mage.constants.SuperType;
 import mage.target.TargetPermanent;
 import mage.target.common.TargetEnchantmentPermanent;
 
@@ -41,8 +43,10 @@ public final class GlissaSunslayer extends CardImpl {
 
         // Whenever Glissa Sunslayer deals combat damage to a player, choose one —
         // • You draw a card and you lose 1 life.
-        Ability ability = new DealsCombatDamageToAPlayerTriggeredAbility(new DrawCardSourceControllerEffect(1).setText("you draw a card"), false, false);
-        ability.addEffect(new LoseLifeSourceControllerEffect(1).concatBy("and"));
+        Ability ability = new DealsCombatDamageToAPlayerTriggeredAbility(
+                new DrawCardSourceControllerEffect(1, true), false, false
+        );
+        ability.addEffect(new LoseLifeSourceControllerEffect(1).setText("and lose 1 life"));
 
         // • Destroy target enchantment.
         Mode mode = new Mode(new DestroyTargetEffect());
@@ -65,4 +69,4 @@ public final class GlissaSunslayer extends CardImpl {
     public GlissaSunslayer copy() {
         return new GlissaSunslayer(this);
     }
-}
+}

Mage.Sets/src/mage/cards/i/Impulsivity.java

@@ -11,7 +11,8 @@ import mage.cards.CardSetInfo;
 import mage.constants.CardType;
 import mage.constants.CastManaAdjustment;
 import mage.constants.SubType;
-import mage.filter.StaticFilters;
+import mage.filter.FilterCard;
+import mage.filter.common.FilterInstantOrSorceryCard;
 import mage.target.common.TargetCardInGraveyard;
 
 import java.util.UUID;
@@ -21,6 +22,8 @@ import java.util.UUID;
  */
 public final class Impulsivity extends CardImpl {
 
+    private static final FilterCard filter = new FilterInstantOrSorceryCard("instant or sorcery card from a graveyard");
+
     public Impulsivity(UUID ownerId, CardSetInfo setInfo) {
         super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{6}{R}");
 
@@ -33,7 +36,7 @@ public final class Impulsivity extends CardImpl {
         Ability ability = new EntersBattlefieldTriggeredAbility(
                 new MayCastTargetCardEffect(CastManaAdjustment.WITHOUT_PAYING_MANA_COST, true)
         );
-        ability.addTarget(new TargetCardInGraveyard(StaticFilters.FILTER_CARD_INSTANT_OR_SORCERY));
+        ability.addTarget(new TargetCardInGraveyard(filter));
         this.addAbility(ability);
 
         // Encore {7}{R}{R}

Mage.Sets/src/mage/cards/w/WickersmithsTools.java

@@ -1,7 +1,5 @@
 package mage.cards.w;
 
-import java.util.UUID;
-
 import mage.abilities.Ability;
 import mage.abilities.TriggeredAbilityImpl;
 import mage.abilities.common.SimpleActivatedAbility;
@@ -22,8 +20,9 @@ import mage.game.events.GameEvent;
 import mage.game.permanent.Permanent;
 import mage.game.permanent.token.ScarecrowToken;
 
+import java.util.UUID;
+
 /**
- *
  * @author muz
  */
 public final class WickersmithsTools extends CardImpl {
@@ -38,7 +37,10 @@ public final class WickersmithsTools extends CardImpl {
         this.addAbility(new AnyColorManaAbility());
 
         // {5}, {T}, Sacrifice this artifact: Create X tapped 2/2 colorless Scarecrow artifact creature tokens, where X is the number of charge counters on this artifact.
-        Ability ability = new SimpleActivatedAbility(new CreateTokenEffect(new ScarecrowToken(), new CountersSourceCount(CounterType.CHARGE)), new ManaCostsImpl<>("{5}"));
+        Ability ability = new SimpleActivatedAbility(new CreateTokenEffect(
+                new ScarecrowToken(), new CountersSourceCount(CounterType.CHARGE)
+        ).setText("create X tapped 2/2 colorless Scarecrow artifact creature tokens, " +
+                "where X is the number of charge counters on {this}"), new ManaCostsImpl<>("{5}"));
         ability.addCost(new TapSourceCost());
         ability.addCost(new SacrificeSourceCost());
         this.addAbility(ability);

Mage/src/main/java/mage/abilities/dynamicvalue/common/ColorsAmongControlledPermanentsCount.java

@@ -45,7 +45,7 @@ public enum ColorsAmongControlledPermanentsCount implements DynamicValue {
 
     @Override
     public String getMessage() {
-        return "for each color among " + filter.getMessage();
+        return "color among " + filter.getMessage();
     }
 
     @Override