Add support for optional progress step bar labels

[hartsick]

Finishes #105

Usages:

    <!-- With default step name ("Step") -->
    <%= render partial: 'components/molecules/progress_step_bar', locals: { current_step: 3, step_count: 5, step_description: 'Basic Info' } %>

    <!-- With custom step name ("Section") -->
    <%= render partial: 'components/molecules/progress_step_bar', locals: { current_step: 3, step_count: 5, step_name: 'Section', step_description: 'Basic Info' } %>

[tdooner]

lgtm, two pieces of unsolicited advice here.... and you may wish to get another review if you're looking for any specific context-based comments.

[tdooner]

a nitpick: local_assigns[:step_description].blank? is marginally better for checking text presence

[hartsick]

good point! wasn't thinking about programmatically rendering a step description (imagine people will manually provide) when I wrote

[tdooner]

for components partials, I've seen a pattern of all local_assigns usages at the top, so users of the component can see all the parameters. I don't know if we want to adopt that pattern but if we did it would include step_description up here as well... e.g.

<% step_name = local_assigns[:step_name] || 'Step' %>
<% step_description = local_assigns[:step_description] %>

(I think the styleguide example is sufficient here, this is just a thought)

[hartsick]

I like that pattern suggestion!

[hartsick]

lgtm, two pieces of unsolicited advice here.... and you may wish to get another review if you're looking for any specific context-based comments.

this was very much solicited feedback. thanks for taking a look @tdooner !