Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Form field tel using JLayouts #13435

Merged
merged 9 commits into from Jan 5, 2017
Merged

Form field tel using JLayouts #13435

merged 9 commits into from Jan 5, 2017

Conversation

dgrammatiko
Copy link
Contributor

@dgrammatiko dgrammatiko commented Dec 31, 2016

Pull Request redo of #10994

Separate logic/ output

Base work for any serious work for template update etc..

Summary of Changes

Introduce a layout for this field

Testing Instructions

Apply patch and rename any backend input to tel
eg

<field name="mytextvalue" type="tel" default="Some text" label="Enter some text" description="" size="10" />

Also consult: https://docs.joomla.org/Standard_form_field_types


This change is Reviewable

@ghost
Copy link

ghost commented Jan 3, 2017

I have tested this item ✅ successfully on 81ece3c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13435.

@anibalsanchez
Copy link
Contributor

I have tested this item ✅ successfully on 81ece3c

Test OK


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13435.

@jeckodevelopment
Copy link
Member

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13435.

@joomla-cms-bot joomla-cms-bot added PR-staging Unit/System Tests RTC This Pull Request is Ready To Commit labels Jan 4, 2017
@jeckodevelopment jeckodevelopment added this to the Joomla 3.7.0 milestone Jan 4, 2017
@rdeutz rdeutz merged commit fab9fae into joomla:staging Jan 5, 2017
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 5, 2017
@dgrammatiko dgrammatiko deleted the +Field_tel branch January 5, 2017 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants