? Pending

User tests: Successful: Unsuccessful:

avatar OctavianC
OctavianC
27 Apr 2017

Pull Request for Issue # .

Summary of Changes

placeholder="" attribute doesn't work on new calendar field due to missing echo.

Testing Instructions

Call JHtml::_('calendar') with a 'placeholder' => 'test' in the $attribs array, eg:

JHtml::_('calendar', $value, $name, $id, $format = '%Y-%m-%d', $attribs = array('placeholder' => 'test'));

Expected result

placeholder="test" in the HTML output.

Actual result

No placeholder attribute shown in HTML.

Documentation Changes Required

None

avatar OctavianC OctavianC - open - 27 Apr 2017
avatar OctavianC OctavianC - change - 27 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Apr 2017
Category Layout
avatar Quy
Quy - comment - 27 Apr 2017

Duplicate of #15427 except this pr escaped the placeholder value.

avatar wilsonge wilsonge - change - 27 Apr 2017
Labels Added: ?
avatar PhilETaylor
PhilETaylor - comment - 27 Apr 2017

see #15633 - this is just one of many that needs doing, I recommend closing this in favor of #15633

avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Apr 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-04-29 06:05:13
Closed_By franz-wohlkoenig
avatar joomla-cms-bot
joomla-cms-bot - comment - 29 Apr 2017
avatar joomla-cms-bot joomla-cms-bot - close - 29 Apr 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 29 Apr 2017

closing this in favor of #15633


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

Add a Comment

Login with GitHub to post a comment