Refactors to BEM and cleans up some markup

[rauhryan]

Relates to #358

I jumped into the comments on #358 and wanted to help out.

I started off by finding a rough closed div tag inside the form tags and started backtracking and folding up all the markup to find out how to fix it. I then needed to add indenting so I'm very sorry for the diff not being very useful.

Things I have address in the PR

• Fixes href quoting problem
• Check if the repo owner is an organization
• Adds analytic paths to determine retrys or abandons

[discorick]

I am curious if you have a GA goal workflow in mind for abandons, we can infer them currently by any traffic that doesn't meet the goal requirements. Having said that 👍 for more data :D

[discorick]

Not a fan of the big purple buttons ?

[rauhryan]

Idk based on the current implementation

you'd see

• /board/create => /board/creating
• /board/create => /board/creating/abandon
• /board/create => /board/creating => /board/creating/retry
• /board/create => /board/creating => /board/creating/abandon

Or some infinity combination of /creating => (retry...abandon)

[rauhryan]

Not a fan of the big purple buttons ?

It wasn't the design that threw me off, I actually prefered the bigger purple buttons.

The css debt didn't seem worth it, there was a triple specific selector on a 'input.btn.primary' nested under another class

There was also use of forms and form inputs inside anchor tags. They didn't really need to be forms and serve just fine as anchor tags that are styled.

I got carried away fixing the markup that I just simplified it all to use the built in bootstrap styles.

[discorick]

It wasn't the design that threw me off, I actually prefered the bigger purple buttons.

The css debt didn't seem worth it, there was a triple specific selector on a 'input.btn.primary' nested under another class

Not sure I 100% follow - if we prefer a certain styling, why can't we achieve without the debt ? Aren't we optimizing in the opposite direction of where we want to go then?