Yeah, I thought the jokes fit the film very well. The only thing I didn't think worked was just the overall scope of Asgard didn't seem big enough. We're talking about an entire planet with an ancient race of beings. And, yet, the population seems to be about the size of a small town, residing in the space of a neighborhood. It just didn't feel like an entire civilization was being wiped out by Hela (with Surtur finishing the job). As a result, the emotional stakes were far lower than they should have been, given the actual conflict of the film. Somehow, it still mostly works despite that. But that flaw bothers me. They could easily have made the scope bigger without overly extending the film. Just have Hela summon an army that helps her preliminarily take the capitol, and then she is able to go into the vaults to resurrect the rest of her army. Then she goes on a tear, and we get a montage of death and destruction around the planet, with Asgardians fleeing and hiding in various places. Then, at the end, have a small fleet of survivors escape from a few locations around the planet, instead of just one ship. It (and the things they obviously have planned at the beginning of Infinity War) would still work, but we get a movie where the stakes are appropriately raised.