Skip to content

Feat - Add card clues to narrow down the card - #116

Closed
daryllxd wants to merge 4 commits into
scryfall:mainfrom
daryllxd:feat/add-card-clues
Closed

Feat - Add card clues to narrow down the card#116
daryllxd wants to merge 4 commits into
scryfall:mainfrom
daryllxd:feat/add-card-clues

Conversation

@daryllxd

@daryllxd daryllxd commented May 20, 2025

Copy link
Copy Markdown

Hello, I am a Magic player that has recently changed jobs and is looking to learn more Vue.js. I accidentally found this repo while I was looking to create some tools for Magic the Gathering and this is just my first PR to add clues because while I was testing the art game, I really couldn't figure out any of the cards but the clues kind of helped. I would like to have a few weeks until my new job starts so I can put in a few hours a day to contribute.

I can also pick one from the issues if it's fine

CleanShot.2025-05-20.at.22.58.52-converted.mp4

import CardArt from "./CardArt.vue";
import GuessInput from "./GuessInput.vue";
import { isGuessOk } from "../utils/guess";
import CardArt from "./CardArt.vue";

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, maybe my linter is not set up correctly (I am using Cursor)

set: string;
image_uris: ScryfallImageUris;
card_faces?: never;
cmc: number;

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is duplicated for now and I notice this exists https://github.com/scryfall/api-types, checking if this can be in another PR or in this one?

align-items: stretch;
max-height: calc((var(--count) + 0.6) * (1em + var(--v-padding) * 2));
overflow-y: scroll;
z-index: 10;

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes this issue, admittedly a bit haphazard since there is no z-indexes.ts yet, which we can add in this PR too.
CleanShot 2025-05-20 at 22 46 42

@daryllxd daryllxd changed the title Add card clues Feat - Add card clues to narrow down the card May 20, 2025
@crookedneighbor

Copy link
Copy Markdown
Collaborator

Hey! Didn't see this until just now. We're in the process of switching to a different store library (PR here) - once that's merged in, and this PR has been updated to use the new system, I can take a look at this PR.

@scarletcs

Copy link
Copy Markdown
Collaborator

Since this PR opened, I've added a CONTRIBUTING.md file laying out our guidelines. While I've considered adding a hint system of some kind, we're not currently looking to add this hint system here.

@scarletcs scarletcs closed this Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants