[#34091] - [4.0] Layout for GroupedlistField
- Fixed in Code Base
- 25 May 2021
- Medium
- Build: 4.0-dev
- # 34091
- Diff
- Fedik:groupedlist
User tests: Successful: Unsuccessful:
Pull Request for Issue #31249 and #33183 .
Summary of Changes
Move GroupedlistField::getInput() to the layout.
Available 2 layout:
joomla.form.field.groupedlist - default, a regular select input
joomla.form.field.groupedlist-fancy-select - optional, fancy select
Testing Instructions
Create 2 field, with different layout, somewhere:
<field
name="field0" type="templatestyle" label="Field 0"
multiple="multiple"
/>
<field
name="field1" type="menuitem" label="Field 1"
multiple="multiple"
layout="joomla.form.field.groupedlist-fancy-select"
/>Actual result BEFORE applying this Pull Request
Both fields will show default <select> input
Expected result AFTER applying this Pull Request
First field will be default <select> input
Second field will be "fancy select"
Documentation Changes Required
Yes, update doc about Groupedlist field.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Layout Libraries Front End Plugins |
| Labels |
Added:
?
|
||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34091.
The test result from @Kostelano is still valid since the change after that was only code style with spaces and the year in the copyright statements.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34091.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-25 20:43:40 |
| Closed_By | ⇒ | chmst | |
| Labels |
Added:
?
|
||
Thanks!
I have tested this item✅ successfully on a6916f8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34091.