Roolo
Forum Replies Created
-
Forum: Plugins
In reply to: [Advanced Custom Fields: Image Crop Add-on] Update for WP 4.4.1Nevermind, updating to the latest ACF vervion resolved the issue. I really thought i had v5.3.6.1 installed locally, maybe they just released some hotfixes without changing the version number?
Forum: Plugins
In reply to: [Advanced Custom Fields: Image Crop Add-on] Update for WP 4.4.1Thanks for your reply. Strange that you couldn’t reproduce it. Did you try cropping an image that was already added or did you try to add a new image using the acf crop field? Cropping images that are already added works, but the problem arises when you try to add an image using a acf crop field.
I don’t think it’s is browser or environment related. I think problem is that the acf.get_field_key function does not exist in ACF pro 5.3.6.1, hence the error:
Uncaught TypeError: acf.get_field_key is not a functionI solved it locally by commenting out line 56 in input.js that uses the function and the plugin works as expected again after that when uploading one image at a time. I don’t know what are the consequences of removing that line though so i don’t know if thats a viable solution.
Forum: Plugins
In reply to: [Advanced Custom Fields: Image Crop Add-on] Update for WP 4.4.1I’m also having this problem, I think the problem is related to the lated acf (pro) update, and not the worpress version.
Yes I think that would be an improvement. The only problem is that the possibility for a switch is then hidden for most users. A way to make it non-hidden is to add an option to the plugin settings:
– add toggle to grid view in media assistant (only works when disabling the default media library).Now the user is informed, and will know the consequence of enabling the default view + enabling the toggle button in media assistant.
Thanks. The url’s are generated right now. For me this is a viable solution, but i can imagine it isn’t for general users. If users don’t disable the default media view, the navigation (Assistant->media/grid->media/list) is still a bit confusing in my opinion.
Another small issue is that the Assistant menu item isn’t highlighted when the grid view is selected and the default media view is disabled.
Thanks! It’s good to see this plugin is so actively maintained.
I’ve just tested the new switch, but unfortunately the url’s seem to be wrong.
For example for the grid view the url is:
http://localhost/wp-admin/upload.php?mode=grid
while it should be (in my case):
Thanks a lot for your quick and clear reply.
I might be able to add a list/grid toggle to the Media/Assistant submenu
This solution could be helpful. But if you navigate from Media/Assistant list to Media/Library grid, how would you navigate back to the Media/Assistant view? The Media/Library ‘toggle to list’ button would take you back to the Media/Library list view, which would be even more confusing.
I understand reïmplementing the grid view in Media/Assistant would be hard, but wouldn’t expanding the MMMW in the Media/Library list view, like you did with the Media/Library grid view be a solution? You could make it an option just like you did with the grid view.