Regarding Harry and Ginny: This was more profound to me than any of the other romantic parts of the movie. Ginny got down and tied Harry’s shoe. This was powerful to me. It would be corny if observed by another, but they had privacy. This was a powerful and intimate gesture by Ginny. It was more powerful than the kiss. This was an act of devotion. I thought it was well done.
In the book, after Dumbledore was killed and Harry was standing by his body, Hagrid couldn't get Harry to leave, and Ginny just walked up and took his hand and led him away. It was a little touch that showed the thought she put into her characters.
OTOH, Rowling has some of the most absurd plot holes imaginable. The one that's always gotten me is that they selected Pettigrew as the secret keeper. I thought the secret keeper had to be someone who didn't live at the house, but Bill was the secret keeper for his and Fleur's house. So why didn't James Potter just be the secret keeper for his house?
In Prisoner of Azkaban, it never occurred to Dumbledore or Snape to check the tunnel Dumbledore had set up for Sirius' best friend to see if that was how he was getting into the castle?
In Goblet of Fire, they spent nine months trying to get Harry to win the TriWizard Tournament so he could touch the TriWizard Cup and get transported to Voldemort when they could have made anything a portkey and transported him anytime during the term? And how come the TriWizard Cup was the only portkey that took you back when you touched it again? None of the other portkeys did that. Oh, and what was with the TriWizard events, anyway? The crowd is supposed to be excited about looking at the surface of the lake for an hour, or watching the outside of a hedge? How come, except for the dragons, all the events had an audience, but the audience couldn't see what the competitors were doing?