Skip to content

Bootstrap admin panel tabular fix#771

Merged
twalen merged 30 commits into
sio2project:masterfrom
malevitzch:bootstrap-update
Jun 24, 2026
Merged

Bootstrap admin panel tabular fix#771
twalen merged 30 commits into
sio2project:masterfrom
malevitzch:bootstrap-update

Conversation

@malevitzch

Copy link
Copy Markdown
Member

Bootstrap 5 made the 'row' class add negative margins which caused certain parts of the UI to be almost unusable. Removing row seems to fix the issue without breaking anything.

Before and after:
image

image

malevitzch and others added 29 commits April 3, 2026 12:10
…d badge-success, bootstrap 5 uses text-bg-success and text-bg-danger etc
…mping jquery to 4 causes problems with webpack expose-loader, this needs further investigation
Fixed My Submissions and Contest tops not being rounded
@malevitzch malevitzch requested a review from MasloMaslane as a code owner June 21, 2026 19:11
@malevitzch

Copy link
Copy Markdown
Member Author

@otargowski I think this was one of the things annoying you, removing the 'row' in tabular.html makes the UI usable

@otargowski

Copy link
Copy Markdown
Contributor

Looks great now! oioioi/base/templates/nesting/admin/inlines/tabular.html should probably be fixed in a similiar fashion.

@malevitzch

Copy link
Copy Markdown
Member Author

Ruff randomly failed for me here with error code 404 so I had to rerun it... not sure if it's an issue on our part, it just failed to access the action so it's probably on astral's side.

Anyways, the other fix is now applied too

@twalen twalen merged commit 8873c03 into sio2project:master Jun 24, 2026
4 of 5 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants