search

beta-nu-theta-chi / ox-dashboard

A chapter dashabord web app in Django for the Beta Nu Chapter of Theta Chi at Case Western Reserve University
MIT License
0 stars 0 forks source link

Refactor templates #14

Open mtbentley opened 7 years ago

mtbentley commented 7 years ago

TODO: write a todo list for this

mtbentley commented 7 years ago

The biggest pain point for me right now is forms. Most of them are the same, while only changing the title between templates. These could be combined, and require passing a title or similar variable in the context.

DLance96 commented 7 years ago

Now that Im finally home I should be able to work on it.

mtbentley commented 7 years ago

No rush. I'm just trying to get my thoughts down, and we still have a couple weeks.

mtbentley commented 7 years ago

Another thing: automatically show the dropdown for a position when the logged in user holds that position.

DLance96 commented 7 years ago

Im finishing up forms not sure that is the best idea if someone has 2 chair positions their top bar could get filled quickly just a thought

mtbentley commented 7 years ago

Hmm, ok. At least split the menu part into a common file that gets included on each page for that user.

DLance96 commented 7 years ago

The way it currently is that it is section in the base.html how is that exactly different?

mtbentley commented 7 years ago

Maybe I've been doing it wrong? IE for the detail manager, the detail manager dropdown shows up on the detail manager page, but not any of the detail manager sub-pages.

DLance96 commented 7 years ago

so you should be able to make a detail manager template and have all the detail manager pages extend that it was something I had planned on doing with other positions once I realized I shouldve done it in the first place. If you have a better structure let me know.

mtbentley commented 7 years ago

Yep of course I should have done that with the detail manager. I'll fix that in a bit.