-
Notifications
You must be signed in to change notification settings - Fork 0
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
Old (SS4) design was easier to use. #4
Comments
@asecondwill did you run this from the main branch? #1 contains a major rework of the front-end of this module which should resolve most UI/UX concerns. |
@asecondwill thanks for the feedback. The UI is indeed broken in
It is kinda hard to see on your screenshot, which debug bar do you mean? Or do you mean that you are running an other module aswell and that conflicts with the admin toolbar? |
its this: https://github.com/lekoala/silverstripe-debugbar which is the grey square behind your bar on the right there |
Thanks, I'll discuss this with our developers. At first glance I'm not sure whether there is a clear solution since we're "fighting" for the same real estate which makes sense because the bottom is imo the proper location for this kind of toolbars, so moving it back to the top by default just to resolve a conflict with a development module seems like a bad decision. |
@asecondwill we feel that it is worth to look into a solution for the conflict with this module and have a couple of idea's on how to resolve it. I do not know yet when we'll land the patch for that but it will most likely be resolved before we'll release the first tagged version of a silverstripe 5 compatible release. |
The new menu based system is a bit fiddly. the bar accross the top was nice and simple in previous versions.
The text was updated successfully, but these errors were encountered: