-
Notifications
You must be signed in to change notification settings - Fork 3
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
Broken on Craft Pro 3.7.31 and newer #7
Comments
Confirmed here (using Craft 3.7.34 and Matrix Toolbar 1.0.6). Collapse doesn't do anything. No issues mentioned in the console. Hopefully this could get some attention! Without this functionality it's pretty difficult to work with huge matrix fields with long blocks! Thanks in advance! |
Having the same problems here. Would love to see a fix! Thanks in advance, this is a very helpful plugin. |
Same issue! |
Just PR’d a fix for this (#9). You can use it in your project by adding this to {
"minimum-stability": "dev",
"prefer-stable": true,
"require": {
"...": "...",
"monachilada/craft-matrixtoolbar": "dev-fix-7"
},
"repositories": [
{"type": "git", "url": "https://github.com/brandonkelly/craft-matrixtoolbar.git"},
]
} |
Thanks @brandonkelly but I'm not able to get the your plugin into the project. I have this in my
I even tried it without the
|
The You probably need to add the following though: {
"minimum-stability": "dev",
"prefer-stable": true,
} |
Ahh it looks like for w/e reason the JS updates made got wiped some how. Having a case of the Mondays 😹 |
@brandonkelly I'm not sure what's happening now, but composer is having issues finding your patch:
Yet while running Do you have any pointers? Thanks! |
@DynamiteGoesBoom sorry! My composer.json example had a typo. |
I tried that too already 🤷♀️
|
@brandonkelly When I made that
I haven't ran into this previously and not sure why that's happening. Thanks for any insight on this and the help with the patch is really helpful! |
@DynamiteGoesBoom Probably due to a rate limitation on the GitHub API. Did you try following the instructions in that message? |
@brandonkelly Yeah I did. I've never ran into this issue with any other repos before so I'm sort of at a loss on this one :/ After adding a token I receive this error: Any ideas on why that new error appears? |
@DynamiteGoesBoom @brandonkelly Just tested it here too, but eventhough I don't get to see the errors as shown above here, the plugin still doesn't work. It's sad, because 'collapse all' and 'expand all' are basic features IMO that cannot be missed as we're working with lots of blocks per page and it's a lot of work to select them all only to collapse or open them. @brandonkelly would it be possible to, instead of having a plugin for this basic functionality, include these features for matrix blocks standard in Craft 4? I wouldn't expect these features to be very difficult to implement, but they add so much to the user control. And then we don't have the issue of this developer not reacting anymore solved too! That would be great! Going to remove this plugin from all our projects here now tho, as this developer obviously doesn't responded for a while now. Hope he/she/they are all good tho. |
For future reference: just added a discussion to implement the 'Collapse all' and 'Expand all' to the core of Craft 4: craftcms/cms#11531 |
@brandonkelly Is there any update on this plugin? Like @bencresty said this would be an excellent core feature for Craft. I'd like to think that it wouldn't be too crazy to implement either. Thanks again for all that you and the team does to make our lives that much easier :) |
@DynamiteGoesBoom Not my plugin so 🤷🏻 You can follow along at craftcms/cms#11531 for any updates pertaining to this making it into Craft. |
Thanks @brandonkelly I'm following that thread now and thanks 👍 |
Hello there!
First of all, thank you for creating this plugin! However, it looks like the Collapsing/Expanding functionality no longer works on
Craft Pro 3.7.31
. I can use the Select All toggle to select all of the matrix blocks, but neither the Collapse or Expand functionality works. Also theSet Status
doesn't work either. It's likely due to how Craft updated the selectors or something recently.Update
I have confirmed rolling Craft back to 3.7.30 corrects the issue. So the issue is certainly with 3.7.31 of Craft.
it's also broken on Craft 3.7.32 as well :(
Thanks again for the help!
The text was updated successfully, but these errors were encountered: