Skip to content
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

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

beelauuu
Copy link
Contributor

@beelauuu beelauuu commented Jul 9, 2023

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

@nsandler1 nsandler1 changed the title Fix for Issue #130 Center table on course difficulty page Jul 12, 2023
@nsandler1
Copy link
Member

nsandler1 commented Jul 12, 2023

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?

@beelauuu
Copy link
Contributor Author

@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!

@nsandler1
Copy link
Member

nsandler1 commented Jul 12, 2023

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?

@beelauuu
Copy link
Contributor Author

@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!

@beelauuu
Copy link
Contributor Author

@nsandler1 never responded about this haha whoops, but I think it looks better with it all centered.

@tybug
Copy link
Contributor

tybug commented Dec 30, 2023

I'm not sure I'm a fan of this, tbh. Compare:

master this pr
image image

I think having the search bar and page selector be centered looks weird, but left-aligning them would also probably look weird when everything else is centered.

@beelauuu
Copy link
Contributor Author

beelauuu commented Dec 30, 2023

Gotcha, what do you think I should do then? I agree they both look a little off (centered and left aligned)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Center table on course difficulty page
3 participants