Conversation
|
I like the layout improvements! Though I do find the font quite hard to read - the kerning for bold fonts and the smaller font size really reduces readability. Could you choose something a little cleaner? |
|
Generally, I would avoid FontWeight Bold with m6x11plus, it should look great without Bold enabled, even in small fonts. EDIT: I wonder if removing this bold would make it look better: Line 495 in 780c1c2 |
|
Thanks for your comments, and thanks for giving it a try. I tried not to change anything related to fonts or general styling decisions, other than maybe the path of the file. In UI.js there's a Lines 494 to 495 in 8343999 The only place it's used bold, is the queue title (above). You can see the difference below between normal and bold options: Untitled.mov |
OptimusPi
left a comment
There was a problem hiding this comment.
I have no authority here but.. I like it :-) It works well the the deployed site.
Hi,
First of all, great idea! Thanks for making it available.
After playing with it a bit, I did some visual changes to make things more.. aligned. Didn't want to change the code that much from original, so didn't change the folder structure, or separated css/js into different files. But simplified a few things along the way.
Visual changes:
I also considered using the full width of the row for inputs (or share it like 1/3 and 2/3 with labels) but on a larger screen, that becomes a disturbingly long field. This felt more aligned, when they end at the same place as buttons below.
Functional changes:
1.0.1o-fullversion, since i was touching the file. closes Added version 1.0.1o-full #10 thanks a lot to @jeroenvervaeke for this.I hope this makes sense. In case someone wants to play with a live version, I deployed my repo to Vercel to see if it's working as expected (with wasm and all): https://balatro-seeds.vercel.app
Cheers,
G.