Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Use phone_field helper for semantic markup. #1556

Closed
wants to merge 1 commit into from

2 participants

@JDutil
Collaborator

No description provided.

@schof
Owner

Good idea!

@schof schof closed this in 55bacb2
@tvdeyen tvdeyen referenced this pull request from a commit in magiclabs/spree
@JDutil JDutil Use phone_field helper for semantic markup.
[Fixes #1556]
7f573d1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on May 17, 2012
  1. @JDutil
This page is out of date. Refresh to see the latest.
View
2  core/app/views/spree/admin/shared/_address_form.html.erb
@@ -71,7 +71,7 @@
<td class="lbl-col">
<%= f.label :phone, t(:phone) + ':' %>
</td>
- <td colspan="7"><%= f.text_field :phone %></td>
+ <td colspan="7"><%= f.phone_field :phone %></td>
</tr>
</tbody>
</table>
View
8 core/app/views/spree/checkout/_address.html.erb
@@ -68,12 +68,12 @@
</p>
<p class="field" id="bphone">
<%= bill_form.label :phone, t(:phone) %><span class="req">*</span><br />
- <%= bill_form.text_field :phone, :class => 'required' %>
+ <%= bill_form.phone_field :phone, :class => 'required' %>
</p>
<% if Spree::Config[:alternative_billing_phone] %>
<p class="field" id="baltphone">
<%= bill_form.label :alternative_phone, t(:alternative_phone) %><br />
- <%= bill_form.text_field :alternative_phone %>
+ <%= bill_form.phone_field :alternative_phone %>
</p>
<% end %>
</div>
@@ -155,12 +155,12 @@
</p>
<p class="field" id="sphone">
<%= ship_form.label :phone, t(:phone) %><span class="req">*</span><br />
- <%= ship_form.text_field :phone, :class => 'required' %>
+ <%= ship_form.phone_field :phone, :class => 'required' %>
</p>
<% if Spree::Config[:alternative_shipping_phone] %>
<p class="field" id="saltphone">
<%= ship_form.label :alternative_phone, t(:alternative_phone) %><br />
- <%= ship_form.text_field :alternative_phone %>
+ <%= ship_form.phone_field :alternative_phone %>
</p>
<% end %>
</div>
Something went wrong with that request. Please try again.