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

Password field using JLayout #11003

Closed
wants to merge 3 commits into from
Closed

Password field using JLayout #11003

wants to merge 3 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 password
eg

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

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

@RonakParmar
Copy link

I have tested this item ✅ successfully on 0600cc4


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

1 similar comment
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 0600cc4


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

@brianteeman
Copy link
Contributor

RTC


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

@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 02ccfa2 - Thanks!

@wilsonge wilsonge closed this Jul 30, 2016
@brianteeman brianteeman removed the RTC This Pull Request is Ready To Commit label Aug 2, 2016
@dgrammatiko dgrammatiko deleted the +FieldPassword 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