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

Email form field using JLayout #11009

Closed
wants to merge 2 commits into from
Closed

Email form field using JLayout #11009

wants to merge 2 commits into from

Conversation

dgrammatiko
Copy link
Contributor

@dgrammatiko dgrammatiko commented Jul 3, 2016

Separate logic/ output

Base work for better templating

Summary of Changes

Introduce a layout for this field

Testing Instructions

Apply patch and rename any backend input to email
eg

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

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

@RonakParmar
Copy link

I have tested this item ✅ successfully on 0656f2d


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

1 similar comment
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 0656f2d


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

@brianteeman
Copy link
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 5, 2016
@brianteeman brianteeman modified the milestone: Joomla 3.7.0 Jul 11, 2016
wilsonge pushed a commit that referenced this pull request Jul 30, 2016
@wilsonge
Copy link
Contributor

Merged with b495656 - thanks!

@wilsonge wilsonge closed this Jul 30, 2016
@wilsonge wilsonge removed the RTC This Pull Request is Ready To Commit label Jul 30, 2016
@dgrammatiko dgrammatiko deleted the +FieldEmail branch October 11, 2016 11:33
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.

5 participants