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

admin.widgets.js #8

Merged
merged 1 commit into from Jan 29, 2014
Merged

admin.widgets.js #8

merged 1 commit into from Jan 29, 2014

Conversation

ghost
Copy link

@ghost ghost commented Jan 29, 2014

Rewrote admin.widgets.js to allow external styling. Added drop functionality to available widgets to accept installed widgets drop to delete.

Added two classes to admin.css to style hover colors of available widgets, installed widgets and trashcan.

I think I fixed the bodged push I did yesterday, you shouldn't see any changes other than those three files on master. I am branching master off to do the HTML5 work because I will be doing heavy modifications of the output.

…onality to available widgets to accept installed widgets drop to delete.

Added two classes to admin.css to style hover colors of available widgets, installed widgets and trashcan.

Signed-off-by: liquibyte <liquibyte@gmail.com>
evacchi added a commit that referenced this pull request Jan 29, 2014
merging admin.widgets.js
@evacchi evacchi merged commit 4f78a0d into flatpressblog:master Jan 29, 2014
@evacchi
Copy link
Collaborator

evacchi commented Jan 29, 2014

looks good to me. thanks for contributing! keep it up!

@ghost
Copy link
Author

ghost commented Jan 30, 2014

I forgot to add a change that is required. I did a re-commit with the
file that's corrected. plugins.jquery.php needs to have a more modern
version being called to work correctly. Instead of using local files I
used googleapis.com as it seems to be ubiquitous.

Version 1.7.1 of jquery and 1.8.11 of jquery-ui won't work for this. I
used jquery 1.10.2 and jqueryui 1.10.3 so I think a minor version bump
might be required. I'm sorry for the confusion, I'm new to git and
github so I may make mistakes. I think I'm getting the hang of things
so hopefully I won't make many more.

Michael Grey

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.

1 participant