-
Notifications
You must be signed in to change notification settings - Fork 702
Hide new "skip-to-content" link and nav breadcrumb #803
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
Conversation
With breadcrumbs inside nav, styles not work. Is more easy and retro compatible do this than rewrite saas
|
This repo has not been updated for years as you will see by the many outstanding pull requests. Its better to make contributions to an actively maintained fork - e.g. https://github.com/pulse-mind/django-suit |
|
hI @gamesbook, thanks for the advice but i know the project. I know about Pulse Mind fork, and I tested in my app some days ago, but he adopt some decisions and make some changes i'm not comfortable with, so I decided to fork original repo and make minimal changes that fix the problems I found in my app admin. I know it is unlikely @darklow approve pull request, it's not a problem for me. I use my own fork in the meanwhile. If @darklow approve the pull, yeeeeha! other users will benefit, otherwise, my app admin works as expected. |
Pulse Mind hide changelist-filter. Keep it.
|
I make a mistake, an open the pull request directly from my v2 branch in front o a new branch or commit, so I close the pull request, as I make other changes and I don't want to dirt the pull. |
|
I keep maintaining repo to the level so it doesn't completely fail with newest Django versions and if it is something that can't be fixed in local admin.css |
|
Sincelery, I unknow admin.css option. I'll review it. @darklow This is my initially idea, one pull by issue, with minimal changes easily mergeable. Thanks a lot. |
Fix #800
Fix #801