Refactor Claro's entity-meta stylesheet

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

bspeare → closed merge request !3212

I opened up the MR above when the current one wasn't in a good state. That being said the current one is currently green.

Pretty sure this should be set to Needs Review. I'll review it at some point :D

Could the MR be updated for 10.1? See a lot of changes that don't have to do with claro

Thanks!

Nesting looks good and variable use.

lauriii → closed merge request !3288

I think the code on merge request 3284 looks good. Posted one question on the MR about an open todo comment.

Let's remove the todos, if it's important create an issue for it in the queue.

This is looking really close! We need to 1) make sure that there's an empty line between the code blocks and 2) There's one place where nesting can be used.

I rebased the branch 10.1.x
Please review.

Looks perfect. Thank you!

Hiding the patch file because it looks like the workflow was switched to MR.

The MR is currently blocked according to GitLab and needs to be rebased.

I have rebased the MR. It is ready to merge.

There appear to be a number of unresolved threads on the MR that will need to be resolved before this can be RTBC’d again. Thank you!

Fixed build failure, Also all the feedbacks are addressed. please review

Think this is good now.

lauriii → closed merge request !3284

Committed 243be9a and pushed to 11.x. Thanks!

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