Having a look at the DOC for NGE Nodes, I found a weird behavior where all titles are inverted with their img, resulting in a quite confusing display :
@Tricotou and @Evgeni_Popov, the order of image than title was intentional for a couple of reasons. The first was to give a visual break between the previous listing and the next. The second was to keep the title of the block right next to the opening paragraph about it specifically for the use of screen readers so that the name of the block is read right before the description rather than hearing the name of the block a description about an image of the block and then the introduction to the block. I also wanted the name of the block right next to the intro for people who aren’t using screen readers to cement the name of the block in association with the description. To this end, I styled the anchor links at the top of the page to land on the image so image, then title then description was right at the top of the screen when using the anchors.
With this change, the title of the block is off screen when using the anchors, which is now a bug. I agree there could be some extra spacing after the end of the previous listing to add some visual separation before the image to reinforce that the text has changed to a new block. That change does require a little more effort on our part to make sure that a CSS change does not impact other MD in an adverse way.
I would also request that when you bring a PR for discussion to the forum that you mark the PR as draft. Your PR was already merged with the new behavior of anchor links not working correctly because the entire team wasn’t aware of the topic. I wouldn’t expect anyone reviewing the PR out of context to be automatically aware of the forum topic or any discussion, so we need to help them out by letting them know this is being discussed on the forum. Perhaps with a forum link in the PR as well.
