-
-
Notifications
You must be signed in to change notification settings - Fork 552
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
Fix options menu size #2673
base: master
Are you sure you want to change the base?
Fix options menu size #2673
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
hi! :) @4yman-0 we can start a proper "wishes" here on Github! (CTRL+U) just to make sure, you might knew the 9 buttons used to be equal sizes 3 x 3. |
This comment was marked as outdated.
This comment was marked as outdated.
hi! (done quickly, this also required to set a max-width for full-tabs chrome-extension://bnomihfieiccainjcjblhegjgglakjdd/menu/index.html, which is fine, since more width doesn't help.
/what did you mean? |
starting at #2569: The scrollbar at chrome://extensions/?options=bnomihfieiccainjcjblhegjgglakjdd |
Plz don't just respond to issue comments in the request.
|
This comment was marked as resolved.
This comment was marked as resolved.
@ImprovedTube Any proposals to fix stuff or something (users used to the current layout will be confused). EDIT 1: Remove stuff |
yes, i mean
yes, that's currently unnecessary |
So, we want to let users know "Hey, these 2 buttons are important, the next 4 are OK and last 2 are pretty useless". Probably do some sort of styling that doesn't make it like "Look, a grid!!1!". |
Problem still exists in |
I pulled updates in the OTHER pull request and wow. Look at this error message :)
|
Should fix #2569
TODO:
NOTES:
Unfortunately, https://improvedtube.com/wishes is broken and I can't fix it here.
Edit: I think flexbox is grid now