tedspare / hexle

Wordle for color hex codes. Guess the hex in six tries.
https://hexle-delta.vercel.app
1 stars 0 forks source link

implemented monochrome ui, binary game mechanic change and formatted … #1

Closed DexterStorey closed 1 year ago

DexterStorey commented 1 year ago

A very opinionated ui branch that could use some work / review before merging

Screen Shot 2022-10-22 at 11 59 12 AM Screen Shot 2022-10-22 at 11 57 57 AM
vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
hexle ✅ Ready (Inspect) Visit Preview Nov 4, 2022 at 3:12AM (UTC)
DexterStorey commented 1 year ago

I feel like we need to decide on boldness, roundness and font sizing as part of a design system... I made some arbitrary choices here

DexterStorey commented 1 year ago

should the keys be square? also just realizing the padding's inconsistent between the grid and the keyboard

DexterStorey commented 1 year ago

Also wondering if the indicators on top / bottom would look cleaner as small arrows that hug a bit closer to the char

DexterStorey commented 1 year ago

I think we need to do #5 b4 we come out of draft here

tedspare commented 1 year ago

ITYJ re design system 😅


Also wondering if the indicators on top / bottom would look cleaner as small arrows that hug a bit closer to the char

Core issue here: does an up/high indicator mean "guess higher" or "you're too high"?

DexterStorey commented 1 year ago

LGTM

R u even a FAANG PM

DexterStorey commented 1 year ago

Generally LGTM; nitpicks only

Check it out I think this is ready to go

DexterStorey commented 1 year ago

Looks good on small mobile

image

DexterStorey commented 1 year ago

@tedspare Can you review?

Need help testing the social media preview and Fav coverage on your browsers / socials

Otherwise I think this is good to go- I'll make new prs for styling fine tuning later but I want to merge so I can branch off main for some of the new features

DexterStorey commented 1 year ago

TLDR there's some big ui changes with spacing and sizing - it's very mobile first, but desktop could use a little love when we get a chance

tedspare commented 1 year ago

@DexterStorey LGTM 🚢 🚢 🚢

tedspare commented 1 year ago

Favicon in Arc browser looks good

image

Social preview in Slack is a different color but looks good

image

Lost (I suck at this game so far lol) but didn't get the alert image

Also keyboard doesn't show active: animations anymore - intentional?

DexterStorey commented 1 year ago

Also keyboard doesn't show active: animations anymore - intentional?

removed with the monochrome ui - unless you can think of a way to show active that doesn't involve transparency / color? - it seems too quirky to make it bounce or invert..

Maybe we can make this a new issue about broader monochrome constraints and then think about ways to handle multiple states and notifications with monochrome ui?

DexterStorey commented 1 year ago

We also need to design monochrome ui for alerts / win / loss conditions but I'll make this an issue and self assign

DexterStorey commented 1 year ago

F it I'm merging

This merge closes issues #4 and #5

8 and #9 are created as a result of the breaking changes in this branch