This should have two shop types: craft and second_hand (it is a second_hand shop for craft supplies). The editor wouldn't let me specify two shop tags, and there isn't a single option for the tag value combining both. I left it at shop=craft , but I would appreciate if someone can figure out how to have both at once to better reflect what this shop actually is.

This is honestly the first time I've heard (read) of a "ramp" on a bridge. Even so, Your change seems to have the so-called "ramp" extend MUCH further than it really did: it should only be a foot or two and end almost immediately than extend much further than they were. Also, if this were true, the old bridge was incorrectly added to the map as it does not seem to include any "ramps" added as separate lines.

To add to my previous comment a bit of "proof" that mapbox is outdated: MapBox sti shows the *old* bridge prior to it being demolished, however the old bridge is literally not there (i.e. it has been completely removed, and no trace of it can be seen in the real world). The new bridge could not have started and ended at the road with the old bridge there because the old bridge was in the way of connecting it, however as said above, the new bridge has been finished and does start and end at the intersections.

The bridge DOES start and end at the intersections. The problem with using satelite imagery is that it can become outdate very quickly, as is the case with MapBox: the imagery shows the bridge BEFORE completion, however the bridge has since been completed and now starts and ends at the intersections. Therefore, I must request that you un-revert the changes, otherwise I will redo the change mysef when I have the time to do so.