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

Hidden form field using JLayout #11008

Closed
wants to merge 1 commit into from
Closed

Hidden form field using JLayout #11008

wants to merge 1 commit 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 hidden
eg

<field name="mytextvalue" type="hidden" 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 271fe53


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

1 similar comment
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 271fe53


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

@brianteeman
Copy link
Contributor

RTC


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

@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 c0a5f70 - 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 +FieldHidden branch October 11, 2016 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants