I think we should consider getting the data on the parent and share the data down, so that we may avoid repeatedly fetching the theme data, what do you think? (For another time, not now)

I'm unconcerned about the performance of repeated calls. I would only want to refactor if it makes the code simpler or better since it ultimately doesn't effect the user's experience. That said, I do think that consolidating all of the data fetching logic (into resolvers) and cacheing that so that it's not fetched every time would both improve the code and minimize the data.

I'm unconcerned about the performance of repeated calls. I would only want to refactor if it makes the code simpler or better since it ultimately doesn't effect the user's experience.

Yeah, as you say, my concern is not about performance, but about the repetition of code in multiple children of a component that could do the fetching for them.