-
Notifications
You must be signed in to change notification settings - Fork 39
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
Toolbar install/upgrade bug fixes #247
base: master
Are you sure you want to change the base?
Conversation
I made some changes and I think I left this PR in a partially merged state. Can you fetch the latest from the main repo and help sort that out? I did make a few stylistic changes:
|
I'm not sure if you meant keep the Edit Toolbar text to the right or not, so I pushed up what I had for that atm. I can revert that if you wish (though I think the text works better being centered). Also, note: The old toolbar dropdown layout is self-depreciating due to the fact that any change to the toolbar creates the tools.php file (if it doesn't exist already) that flags the layout as being the new format. I think it'd work best if the fallback (old layout) was left in for the next release, and then removed for the one following. |
Bump |
I did style and move the edit link to the right on purpose. Something about centering it looked off to me... Thinking about the fallback mode, I'm not sure we need it at all since new releases already include a default tools.php file, and IMO the new menu is much nicer now too. |
Done. (Also, might want to check the Travis build... seems a file or class is missing from the tests for php7) |
I'm a bit confused since this was partially merged before and seems to be working well from what I can tell. Can you let me know what the differences are or what's still outstanding? Thanks :) Not sure what's up with the PHP7 issue yet, seems to be that PHPUnit moved some things to external packages and a simple |
…ries Added hardcoded toolbar defaults Changed the action taken when a tools file is unavailable to use, it now generates the file from the defaults Removed the default tools reference file since they are now hardcoded
With the removal of the fallback, if there is no tools file avaliable, it will generate the tools.php file from hardcoded defaults. I also found a bug in the tree-drag-drop script. |
Did some more digging around in this PR and haven't found any other bugs needing squashed. Is there anything else that the rest of this PR needs done or explained? |
Merged the master branch in. Pardon the git stupidity, there was a local file permissions issue that git was complaining about. |
This reverts commit 6915760.
Continuing from #236.
-> Fixed incorrect handler link for tools needing upgrade
-> Fixed "Click to X" duplication bug
-> Added prevention of admin/upgrade being shown in the toolbar editor
-> Added CSS to the toolbar editor to signify if an app requires action taken
Let me know if you wanna have a different label for the admin/upgrade tool.