-
Notifications
You must be signed in to change notification settings - Fork 3
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
Changing category bar on mobile #108
Comments
Wait! This is still broken methinks |
Oops. What was the issue here again? |
Issue is the category bar or the padding aroudn "new listing, ..."? |
I guess both... the whole thing has some wonk. |
Padding is my bad. Thoughts on a good way of dealing with category overflow when the width of the page is not quite short enough to do to the full mobile view? Some thoughts:
|
Yeah, I honestly don't know. The mobile view is already a little weird — so maybe (3)? But then we'd have three regimes rather than just two. |
Yeah I was thinking like a sidebar that comes out when a button is pressed. Was going to do it as part of the polymer migration when I have some time. (Thinking winter break) |
I'm on winter break now with a surplus of free time, are you working on a separate repository for the polymer integration? I would love to help :) |
Right now, I'm focused mostly on switching backends (see PR), but that's definitely something we're considering! Talk to @georgeteo |
@tranv94: Hi! Let's chat if you're still interested in fiddling around with front end changes. We're thinking react + semantic UI. (I've FB messaged you.) |
No description provided.
The text was updated successfully, but these errors were encountered: