Simplify implementation around plots/views

[johny-b]

It's just very confusing now and Claude seems to be confused, could be much easier.

Perhaps we just don't want question.view(MODELS)? It's a complication and .view(df) is almost never worse.

IOW,

question.view(MODELS)

becomes

question.view(question.df(MODELS))

and the code becomes much easier.
We should probably gracefully handle this somehow? Like, if the first argument to Question.view is a dict, have a clear error mentioning this recently changed?

[johny-b]

New idea.

Do this internally:

• check if dict
• if dict, check if class or instance call
• if class call, exception
• if instance call, do df = cls_or_instance.df(arg) and use it later

Use the same for plot.