One point I will make. Elven Accuracy is just +1 Cha unless you're attacking with advantage.

Do you have a plan for how you're going to be attacking with advantage? I've seen a warlock take Elven Accuracy and as of level 12 I think they've used it 2 or 3 times.

It's still fine if you're just taking it for the +1 and don't really care about the other element but you'll need a plan on how advantage is going to be generated if you want consistent use out of the other side.