Add card info tooltips with keyword explanations and related cards

forge-core/src/main/java/forge/util/ImageUtil.java

@@ -260,6 +260,13 @@ public static String getScryfallDownloadUrl(PaperCard cp, String face, String se
             faceParam = (face.equals("back") && cp.getRules().getSplitType() != CardSplitType.Flip
                     ? "&face=back"
                     : "&face=front");
+        } else if (cp.getRules().getSplitType() == CardSplitType.Specialize) {
+            // Specialize faces have their own Scryfall entries with collector
+            // number = base number + color letter (e.g. "2w", "2u", "2b", "2r", "2g")
+            String colorSuffix = specFaceToCollectorSuffix(face);
+            if (colorSuffix != null) {
+                cardCollectorNumber += colorSuffix;
+            }
         }
 
         if (cardCollectorNumber.endsWith("☇")) {
@@ -284,6 +291,17 @@ public static String getScryfallTokenDownloadUrl(String collectorNumber, String
                 langCode, versionParam, faceParam);
     }
 
+    private static String specFaceToCollectorSuffix(String face) {
+        switch (face) {
+            case "white": return "w";
+            case "blue":  return "u";
+            case "black": return "b";
+            case "red":   return "r";
+            case "green": return "g";
+            default:      return null;
+        }
+    }
+
     private static String encodeUtf8(String s) {
         try {
             return URLEncoder.encode(s, "UTF-8");

forge-game/src/main/java/forge/game/keyword/Keyword.java

@@ -16,7 +16,7 @@ public enum Keyword {
     AMPLIFY("Amplify", Amplify.class, false, "As this creature enters, put {%d:+1/+1 counter} on it for each %s card you reveal in your hand."),
     ANNIHILATOR("Annihilator", KeywordWithAmount.class, false, "Whenever this creature attacks, defending player sacrifices {%d:permanent}."),
     ASCEND("Ascend", SimpleKeyword.class, true, "If you control ten or more permanents, you get the city's blessing for the rest of the game."),
-    ASSIST("Assist", SimpleKeyword.class, true, "Another player can pay up to %s of this spell's cost."),
+    ASSIST("Assist", SimpleKeyword.class, true, "Another player can help pay the generic mana cost of this spell."),
     AURA_SWAP("Aura swap", KeywordWithCost.class, false, "%s: You may exchange this Aura with an Aura card in your hand."),
     AWAKEN("Awaken", KeywordWithCostAndAmount.class, false, "If you cast this spell for %s, also put {%d:+1/+1 counter} on target land you control and it becomes a 0/0 Elemental creature with haste. It's still a land."),
     BACKUP("Backup", KeywordWithAmount.class, false, "When this creature enters, put {%1$d:+1/+1 counter} on target creature. If that's another creature, it gains the following ability until end of turn."),
@@ -104,7 +104,7 @@ public enum Keyword {
     HORSEMANSHIP("Horsemanship", SimpleKeyword.class, true, "This creature can't be blocked except by creatures with horsemanship."),
     IMPENDING("Impending", KeywordWithCostAndAmount.class, false, "If you cast this spell for its impending cost, it enters with {%2$d:time counter} and isn't a creature until the last is removed. At the beginning of your end step, remove a time counter from it."),
     IMPROVISE("Improvise", SimpleKeyword.class, true, "Your artifacts can help cast this spell. Each artifact you tap after you're done activating mana abilities pays for {1}."),
-    INDESTRUCTIBLE("Indestructible", SimpleKeyword.class, true, "Effects that say \"destroy\" don't destroy this."),
+    INDESTRUCTIBLE("Indestructible", SimpleKeyword.class, true, "A permanent with indestructible can't be destroyed."),
     INFECT("Infect", SimpleKeyword.class, true, "This creature deals damage to creatures in the form of -1/-1 counters and to players in the form of poison counters."),
     INGEST("Ingest", SimpleKeyword.class, false, "Whenever this creature deals combat damage to a player, that player exiles the top card of their library."),
     INTIMIDATE("Intimidate", SimpleKeyword.class, true, "This creature can't be blocked except by artifact creatures and/or creatures that share a color with it."),
@@ -142,7 +142,7 @@ public enum Keyword {
     PHASING("Phasing", SimpleKeyword.class, true, "This phases in or out before you untap during each of your untap steps. While it's phased out, it's treated as though it doesn't exist."),
     PLOT("Plot", KeywordWithCost.class, false, "You may pay %s and exile this card from your hand. Cast it as a sorcery on a later turn without paying its mana cost. Plot only as a sorcery."),
     POISONOUS("Poisonous", KeywordWithAmount.class, false, "Whenever this creature deals combat damage to a player, that player gets {%d:poison counter}."),
-    PROTECTION("Protection", Protection.class, true, "This creature can't be blocked, targeted, dealt damage, or equipped/enchanted by %s."),
+    PROTECTION("Protection", Protection.class, true, "This permanent can't be blocked, targeted, dealt damage, enchanted, or equipped by %s."),
     PROTOTYPE("Prototype", KeywordWithCost.class, false, "You may cast this spell with different mana cost, color, and size. It keeps its abilities and types."),
     PROVOKE("Provoke", SimpleKeyword.class, false, "Whenever this creature attacks, you may have target creature defending player controls untap and block it if able."),
     PROWESS("Prowess", SimpleKeyword.class, false, "Whenever you cast a noncreature spell, this creature gets +1/+1 until end of turn."),

forge-game/src/main/java/forge/game/keyword/KeywordAction.java

@@ -0,0 +1,168 @@
+package forge.game.keyword;
+
+import forge.util.Localizer;
+
+/**
+ * Keyword actions — verbs that appear in rules text but aren't keyword abilities.
+ * Descriptions are based on the MTG comprehensive rules (section 701).
+ *
+ * <p>Unlike {@link Keyword} (keyword abilities that grant continuous effects or
+ * triggered/static abilities), keyword actions are one-shot game actions performed
+ * when instructed by a spell or ability.</p>
+ *
+ * <p>Actions marked {@code basic=true} are fundamental game actions (destroy, exile,
+ * sacrifice, etc.) that every player knows — UI code may choose to omit these from
+ * tooltips to avoid clutter.</p>
+ *
+ * <p>Display names and reminder text are stored in {@code en-US.properties} under
+ * keys derived from the enum name (e.g. {@code lblKwActionActivate},
+ * {@code lblKwActionActivateReminder}).</p>
+ */
+public enum KeywordAction {
+    // 701.2 – 701.13: Basic game actions
+    ACTIVATE(true),
+    ATTACH(true),
+    BEHOLD(false),
+    CAST(true),
+    COUNTER(true),
+    CREATE(true),
+    DESTROY(true),
+    DISCARD(true),
+    DOUBLE(true),
+    TRIPLE(true),
+    EXCHANGE(true),
+    EXILE(true),
+
+    // 701.14 – 701.17: Combat & graveyard actions
+    FIGHT(false),
+    GOAD(false),
+    INVESTIGATE(false),
+    MILL(false),
+
+    // 701.18 – 701.21: More basic actions
+    PLAY(true),
+    REGENERATE(false),
+    REVEAL(true),
+    SACRIFICE(true),
+
+    // 701.22 – 701.28: Library & transform actions
+    SCRY(false),
+    SEARCH(true),
+    SHUFFLE(true),
+    SURVEIL(false),
+    TAP_UNTAP(true),
+    TRANSFORM(false),
+    CONVERT(false),
+
+    // 701.29 – 701.30: Opponent manipulation
+    FATESEAL(false),
+    CLASH(false),
+
+    // 701.31 – 701.33: Supplemental format actions
+    PLANESWALK(false),
+    SET_IN_MOTION(true),
+    ABANDON(true),
+
+    // 701.34 – 701.36: Counter & token actions
+    PROLIFERATE(false),
+    DETAIN(false),
+    POPULATE(false),
+
+    // 701.37 – 701.39: Creature enhancement
+    MONSTROSITY(false),
+    VOTE(false),
+    BOLSTER(false),
+
+    // 701.40 – 701.44: Face-down & explore
+    MANIFEST(false),
+    SUPPORT(false),
+    MELD(false),
+    EXERT(false),
+    EXPLORE(false),
+
+    // 701.45 – 701.48: Un-set & learning
+    ASSEMBLE(false),
+    ADAPT(false),
+    AMASS(false),
+    LEARN(false),
+
+    // 701.49 – 701.50: Dungeon & connive
+    VENTURE(false),
+    CONNIVE(false),
+
+    // 701.51 – 701.52: Attraction actions
+    OPEN_AN_ATTRACTION(false),
+    ROLL_TO_VISIT(false),
+
+    // 701.53 – 701.54: Incubate & ring
+    INCUBATE(false),
+    THE_RING_TEMPTS_YOU(false),
+
+    // 701.55 – 701.56: Villainous choice & time travel
+    FACE_A_VILLAINOUS_CHOICE(false),
+    TIME_TRAVEL(false),
+
+    // 701.57 – 701.60: Discover, cloak, evidence, suspect
+    DISCOVER(false),
+    CLOAK(false),
+    COLLECT_EVIDENCE(false),
+    SUSPECT(false),
+
+    // 701.61 – 701.64: Bloomburrow & beyond
+    FORAGE(false),
+    MANIFEST_DREAD(false),
+    ENDURE(false),
+    HARNESS(false),
+
+    // 701.65 – 701.68: Avatar & Lorwyn Eclipsed
+    AIRBEND(false),
+    EARTHBEND(false),
+    WATERBEND(false),
+    BLIGHT(false),
+
+    // Game concepts (not 701.x actions, but useful to explain)
+    DEVOTION(false),
+    DOMAIN(false),
+    METALCRAFT(false),
+    THRESHOLD(false),
+    DELIRIUM(false);
+
+    /** True for fundamental game actions (destroy, exile, sacrifice, etc.) that don't need tooltip explanations. */
+    public final boolean basic;
+    /** Translation key prefix derived from enum name, e.g. "lblKwActionActivate". */
+    private final String translationKey;
+
+    KeywordAction(boolean basic) {
+        this.basic = basic;
+        this.translationKey = "lblKwAction" + toCamelCase(name());
+    }
+
+    private static String toCamelCase(String enumName) {
+        StringBuilder sb = new StringBuilder();
+        boolean capitalize = true;
+        for (char c : enumName.toCharArray()) {
+            if (c == '_') {
+                capitalize = true;
+            } else {
+                sb.append(capitalize ? c : Character.toLowerCase(c));
+                capitalize = false;
+            }
+        }
+        return sb.toString();
+    }
+
+    /** Returns the localized display name. */
+    public String getDisplayName() {
+        return Localizer.getInstance().getMessage(translationKey);
+    }
+
+    /** Returns the localized reminder text. */
+    public String getReminderText() {
+        return Localizer.getInstance().getMessage(translationKey + "Reminder");
+    }
+
+    @Override
+    public String toString() {
+        return getDisplayName();
+    }
+}

forge-game/src/main/java/forge/game/keyword/KeywordWithType.java

@@ -19,9 +19,21 @@ public class KeywordWithType extends KeywordInstance<KeywordWithType> implements
 
     @Override
     public String getTitle() {
-        StringBuilder sb = new StringBuilder();
-        sb.append(this.getKeyword()).append(" ").append(descType);
-        return sb.toString();
+        switch (getKeyword()) {
+            case LANDWALK:
+                // "Swampwalk", "Islandwalk", etc. instead of "Landwalk Swamp"
+                return Character.toUpperCase(descType.charAt(0))
+                        + descType.substring(1) + "walk";
+            case AFFINITY:
+                // "Affinity for artifacts" instead of "Affinity artifact".
+                // Some types are already plural (e.g. "Plains"), so skip
+                // pluralisation if descType already ends in "s".
+                final String affinityPlural = descType.endsWith("s")
+                        ? descType : Lang.getPlural(descType);
+                return "Affinity for " + affinityPlural;
+            default:
+                return this.getKeyword() + " " + descType;
+        }
     }
 
     @Override

forge-game/src/main/java/forge/game/keyword/Partner.java

@@ -14,7 +14,7 @@ public String getTitle() {
 
     @Override
     protected void parse(String details) {
-        with = details;
+        with = details.isEmpty() ? null : details;
     }
 
     @Override

forge-game/src/main/java/forge/game/keyword/Protection.java

@@ -10,6 +10,13 @@ public String getTitle() {
 
     @Override
     protected void parse(String details) {
+        fromWhat = details.startsWith("from ") ? details.substring(5) : details;
+        // Details may use "type:description" format (e.g. "Card.MultiColor:multicolored",
+        // "instants:instants") — use the description part for display
+        final int colon = fromWhat.indexOf(':');
+        if (colon >= 0) {
+            fromWhat = fromWhat.substring(colon + 1);
+        }
     }
 
     @Override

forge-game/src/main/java/forge/game/keyword/Trample.java

@@ -4,7 +4,7 @@ public class Trample extends KeywordWithType {
     @Override
     public String getTitle() {
         if (!type.isEmpty()) {
-            return "Trample over planeswalkers";
+            return "Trample Over Planeswalkers";
         }
         return "Trample";
     }

forge-gui-desktop/src/main/java/forge/CachedCardImage.java

@@ -14,7 +14,7 @@ public abstract class CachedCardImage implements ImageFetcher.Callback {
     final int width;
     final int height;
 
-    static final SwingImageFetcher fetcher = new SwingImageFetcher();
+    public static final SwingImageFetcher fetcher = new SwingImageFetcher();
 
     public CachedCardImage(final CardView card, final Iterable<PlayerView> viewers, final int width, final int height) {
         this.card = card;

forge-gui-desktop/src/main/java/forge/ImageCache.java

@@ -40,9 +40,11 @@
 import com.google.common.cache.LoadingCache;
 import com.mortennobel.imagescaling.ResampleOp;
 
+import forge.card.CardStateName;
 import forge.card.CardSplitType;
 import forge.game.card.Card;
 import forge.game.card.CardView;
+import forge.game.card.CardView.CardStateView;
 import forge.game.player.PlayerView;
 import forge.gui.FThreads;
 import forge.gui.GuiBase;
@@ -200,6 +202,10 @@ public static Pair<BufferedImage, Boolean> getCardOriginalImageInfo(String image
         return getOriginalImageInternal(imageKey, useDefaultIfNotFound, null);
     }
 
+    public static Pair<BufferedImage, Boolean> getCardOriginalImageInfo(String imageKey, boolean useDefaultIfNotFound, CardView cardView) {
+        return getOriginalImageInternal(imageKey, useDefaultIfNotFound, cardView);
+    }
+
     // return the pair of image and a flag to indicate if it is a placeholder image.
     private static Pair<BufferedImage, Boolean> getOriginalImageInternal(String imageKey, boolean useDefaultIfNotFound,
                                                                          CardView cardView) {
@@ -336,7 +342,6 @@ private static Pair<BufferedImage, Boolean> getOriginalImageInternal(String imag
                 float screenScale = GuiBase.getInterface().getScreenScale();
                 int width = Math.round(488 * screenScale), height = Math.round(680 * screenScale);
                 BufferedImage art = original;
-                CardView card = ipc != null ? Card.getCardForUi(ipc).getView() : cardView;
                 String legalString = null;
                 original = new BufferedImage(width, height, BufferedImage.TYPE_INT_ARGB);
                 if (art != null) {
@@ -345,7 +350,21 @@ private static Pair<BufferedImage, Boolean> getOriginalImageInternal(String imag
                     int year = cal.get(Calendar.YEAR);
                     legalString = "Illus. " + ipc.getArtist() + "   ©" + year + " WOTC";
                 }
-                FCardImageRenderer.drawCardImage(original.createGraphics(), card, altState, width, height, art, legalString);
+                // For specialize faces, render the specific CardStateView directly
+                // rather than relying on CardView's current/alternate state lookup
+                CardStateView specState = null;
+                if (ipc != null && !specColor.isEmpty()) {
+                    CardStateName specStateName = specColorToStateName(specColor);
+                    if (specStateName != null) {
+                        specState = CardView.getState(Card.getCardForUi(ipc), specStateName);
+                    }
+                }
+                if (specState != null) {
+                    FCardImageRenderer.drawCardStateImage(original.createGraphics(), specState, width, height, art, legalString);
+                } else {
+                    CardView card = ipc != null ? Card.getCardForUi(ipc).getView() : cardView;
+                    FCardImageRenderer.drawCardImage(original.createGraphics(), card, altState, width, height, art, legalString);
+                }
                 // Skip store cache since the rendering speed seems to be fast enough
                 // Also the scaleImage below will already cache re-sized image for CardPanel anyway
                 // if (art != null || !fetcherEnabled)
@@ -367,6 +386,17 @@ private static boolean isWhiteBorderSet(String setCode) {
             setCode.equals("6E") || setCode.equals("7E") || setCode.equals("8E") || setCode.equals("9E");
     }
 
+    private static CardStateName specColorToStateName(String specColor) {
+        switch (specColor) {
+            case "white": return CardStateName.SpecializeW;
+            case "blue": return CardStateName.SpecializeU;
+            case "black": return CardStateName.SpecializeB;
+            case "red": return CardStateName.SpecializeR;
+            case "green": return CardStateName.SpecializeG;
+            default: return null;
+        }
+    }
+
     public static boolean isSupportedImageSize(final int width, final int height) {
         return !((3 > width && -1 != width) || (3 > height && -1 != height));
     }