Viewing this again, it looks to me like the ships arrive from the other side of the forest, which is why they’re skimming the trees where Rey is hiding. I’d suggest giving some more movement to the Star Destroyer, as it is entering the frame from right to left. There’s no need to show the individual fighters and landing craft, since they are all shown later to be traveling fairly horizontally over the forest to the castle, presumably to be a surprise. If everyone saw TIEs and landing craft approaching the castle, there would be immediate pandemonium, but with just the distant Star Destroyer, it’s a bit more believable that there would be a few moments of relative calm beforehand.

In my opinion, the continuity is this: The Star Destroyer enters the system and launches attack crafts before it becomes visible to the castle, and only appears overhead moments before the attack group as a safeguard against escape.

JEDIT: In my ideal version, Finn looks up to see an escaping ship destroyed with lasers, but the source of the lasers is hidden behind the clouds. Then we just start to see the ship come out of the haze and cloud (and possibly the smoke of the explosion) as Finn says that ‘It’s the First Order, they’ve done it!’