Refactor Claro's form--managed-file stylesheet

Gauravvv → made their first commit to this issue’s fork.

.form-managed-file__main has 2 sections that could be merged
form-managed-file__meta has 2 sections that could be merged.

Gaurav-drupal → made their first commit to this issue’s fork.

Addressed #17, please review

Nesting seems much better.

agreed, nesting is nice here. Simply need to remove @nest

Restoring status as we are going ahead with @nest as of now.

You can remove the line and set back to RTBC, appart from that the patch is good to go.

Moving to RBTC per #24

Unaddressed feedback from @nod_, and I added some additional items in the MR. @nod_ is correct that The @nest syntax (literally the thing that says "@nest") should not be used, but you can still nest by placing the ampersand after the selector.

@VladimirAus - I've mentioned this in other issues, but can you refrain from clicking the rebase button if you're not making additional contributions. It makes the issue harder to follow, and anyone making changes to the branch can get things current with HEAD with about 5 seconds work so it doesn't really help advance anything.

I have addressed all the feedbacks. please review

@nod_ correct me if I'm wrong the no @nest thing was when it was discussed moving away from that but that's no longer the case

Points do appear to be addressed.

Rerolling Patch for V11 :)

The last submitted patch, 32: 3303550-32.patch, failed testing. View results →

MR 3571, still applies to D11. There is no need of re-roll. thanks

restoring status.

I left a few bits of feedback on the MR. They should be pretty easy to address though.

Please review.

For the open threads

Addressed all threads, please review

Threads appear to be addressed.

Hiding all the files because we have MRs

lauriii → closed merge request !3571

Committed 86a0d30 and pushed to 11.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.