Closed drusepth closed 10 years ago
Changes made thus far:
Changes thus far are on branch fix-bootplus
Each content type's form and edit_form have been merged (loool, code length cut in half) and redesigned to use fluid-width cards (BS3) rather than a single card with tabs at the top (live).
Pickers are now cleaned up and in a new HtmlHelper. Content for the current user is now grabbed from a new MyContentHelper (rather than continuously calling ContentType.where(user_id: session[:user_id]) everywhere). Probably going to write code to generate forms from a small DSL, because even with half of it, this HTML is still pretty unwieldy.
generate_character_form(fields)
where fields is an array of objects (fields) that look like
[
{
key: string,
type: string (either "textbox" or "textarea"),
linked_content_type: object (nil by default)
},
...
]
Need to figure out how to logically fit the generators in though.
I'd like to mention that LESS has some dependencies that make running this app on Windows very awkward. I have to develop on a Linux virtual machine to get anything done. If you don't think it would be terribly difficult, switching to the SASS version of Bootstrap causes fewer problems. I think developer preferences is leaning toward SASS as well but since you've already got a codebase the effort of the change might outweigh the minor benefits.
SASS and LESS are pretty similar in syntax, right? I think I just went with LESS originally because that is just the bootstrap I grabbed. I don't have a preference one way or the other, but if development preferences are leaning towards SASS I'd be more than happy to pursue migrating from one to the other.
I don't have much experience with either so I can't judge how easy it would be to migrate.
Todo list for blockers on fix-bootstrap
branch
Issues to create for future branches
Small changes to append to existing issues
Wowsers, I think this massive branch is about done. I'll do my best to merge it into master, and hope it causes minimal conflicts when we merge #360 in.
Live.
Currently on Bootstrap 2 with a card CSS library that only supports Bootstrap 2. A lot has changed, this might be pretty extensive, so we might as well make it fun by bundling in a redesign.
The changes associated with this issue are living on branch fix-bootplus.