A few questions:

First of all, I was wondering why you changed the crossings from crossing=uncontrolled to crossing=marked since uncontrolled is the standard tagging for marked crossings these days. from what I can tell the crossing ways and nodes were already tagged correctly. 

Second of all if you look at the latest imagery (Esri World Imagery) the crossing:markings on these ways is ladder not zebra and that is what it was tagged as before this commit so I am wondering why you made those changes.

Happy mapping,
Udar