Skip to content

Conversation

daryllxd
Copy link

@daryllxd daryllxd commented May 20, 2025

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
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
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
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
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
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