-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Desktop UI updates #6
Comments
umm, love it |
LOL looks great! 👏🏻 You may want to left align the text in the groceries list too, just for consistency. I think there are accessibility studies that show it's easier for your eye to traverse through a list if they are left aligned (at least for LTR languages). |
is it? is it jarring to your eye to have the titles centered, but list items left-aligned? |
I would probably left-align them all just out of personal preference, but I think it's less important because you only need to read the titles once. Whereas the list items you might be scanning up and down and so those should be left-aligned for ease of reading. Up to you! |
Hooray, it looks so good! |
@starryeyez024 clear fridge button, in Account (for now) |
Awesome, thank you! Is it on the account settings page just from an ease-of-development perspective for now? |
Yeah, I don't really know where to put it on the Kitchen page, or even if it belongs there (it probably should be on the same page as the list). Also, I figured it wouldn't hinder the UX too much if you had to go to Account before flushing your items. Also, the Kitchen page is already clogged with that super complicated conditional overlay that I want to clean up before adding new actions. |
@starryeyez024 FYI as a sidenote, I want to start decoupling tailwind (I have a LOT of stupid css classes floating around) from this project, and revert everything back to regular CSS. I am also in the process of cleaning up my dependencies, getting rid of headlessui and radix. I feel like making these changes will allow this project to scale a bit better, and also make it easier to contribute to. Would love your thoughts. |
This is not a great mockup, just me messing around in web inspector, but it conveys the idea :)
The text was updated successfully, but these errors were encountered: