-
Notifications
You must be signed in to change notification settings - Fork 22
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
Center table on course difficulty page #133
base: master
Are you sure you want to change the base?
Conversation
Trying to fix things?
No worries, thank you for doing this! I modified your description to close the corresponding issue and update the title of the PR to be a more descriptive. You did exactly what the issue asked of you, but is it me or does something seem off about everything being centered? Maybe keep the search and pagination aligned left or right of the table? What are your thoughts @beelauuu? |
@nsandler1 Hm. I actually just had the table centered at first but thought it looked pretty odd so I just changed some of the bootstrap CSS to center it and thought it looked better all centered If you want I can uncenter the other parts except the table and recommit if you want to see how it looks! |
No need, I trust your judgement. What are your thoughts about making the search bar and pagination thing extend to the full width the of table? |
@nsandler1 Hm off the top of my head I can't really picture it. I'll play around with it tonight and let you know if I can get it to extend all the way and see how it looks! |
@nsandler1 never responded about this haha whoops, but I think it looks better with it all centered. |
Gotcha, what do you think I should do then? I agree they both look a little off (centered and left aligned) |
Centered the tables as well as the searching feature and pagination on /tools/coursedifficulty @nsandler1
EDIT: I am so sorry for my horrible commit history lol. I reused my branch from last time and it became a humongous mess :/
Closes #130