indentation in php files might be a bit wonky because it's being handled by some formatting setting in Nova itself. I may have to make some adjustments to those configs to get it where it needs to be.
Ha, my instincts are reacting to the tables, but that's just old web person trauma.
It's all good for right now. It will probably be changed a few times anyway in the localization process.
The implementation was correct, so that's cool. I just don't use it, so you introduced a new idea that might be a better way to handle views, so I'm just taking a beat to think about the future…
I don't have View Components on the backend as I run everything through Controllers.
I'm trying to decide if that's something I want to implement site wide or keep the same structure that…
I honestly don't know what the best practice is for this scenario, so this works for me.
If we need to change it later, we will.
I was holding off on this because this was a part of the localization effort that I was going to get into soon, but it doesn't hurt to add this now.
I think moving navigation to it's own thing is fine, but this PR also introduces a new paradigm that could have implications down the road.
I'm gonna think about this one for a bit.
Overall, I think these are fine, but I want the focus indicator on the search to be loud. That's the reason for the color flip. If it needs more contrast, I'll work on that, but I don't want…
Ha, I have a deep, philosophical hate of tables, but this makes sense, so it's cool.