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

CSS Stylings possibly break when used with Bootstrap/AdminLTE #19

Open
brodriguez8774 opened this issue May 24, 2022 · 3 comments
Open
Labels
bug Something isn't working

Comments

@brodriguez8774
Copy link
Collaborator

Needs verification on other machines and/or in additional projects.

@brodriguez8774 brodriguez8774 added the bug Something isn't working label May 24, 2022
@DJBarnes
Copy link
Owner

I'd have to check it out myself. I don't see how. The dump screen does not extend from any other templates. This means that the CSS and JS that are loaded should only be what is needed for dumping. Nothing else. So, I don't see how it could conflict.

Unless you are using the template tag. In which case the documentation has a warning that it will collide with the CSS of your site and lead to really weird formatting. It also mentions that the majority of dumping should be done via python code and not the template. The template tag is a "use at your own risk" type of tag.

@brodriguez8774
Copy link
Collaborator Author

I know we verbally discussed this, but just writing it here so it's not forgotten:

This problem only came up once, on a specific machine. If we're unable to replicate this, it's possible that it was something improperly set up on the local machine. Both the django-dump-die and django-adminlte2 packages have also been updated since, so it's possible it was somehow unintentionally resolved from that.

Either way, it seems this is not (currently) a thing that seems easily reproducible. But I vote this issue stays open until both packages are in a more "final" state, at which point we can test again on a few machines (hopefully including the original machine) and see if it can be reproduced at all.

@brodriguez8774
Copy link
Collaborator Author

Have we been able to (or even attempted to) reproduce this since it was last comment? To my knowledge, we have not tried. But probably should, particularly now that both packages are presumably stable, since they're public at this point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants