User tests: Successful: Unsuccessful:
The folderlist form field got a new option to show the list the folders recursive.
<field name="directory" type="folderlist" directory="images" class="input-xxlarge"
required="true" hide_none="true" hide_default="true" recursive="true"
label="COM_FIELDS_FIELD_IMAGELIST_DIRECTORY_LABEL"
description="COM_FIELDS_FIELD_IMAGELIST_DIRECTORY_DESC">
<option value=""></option>
</field>
You should see a new select box which lists all folders recursively which are in the images folder.
Only the folders below the images folders are shown.
Would be great to get that into 3.6 as it is needed for joomla-projects/custom-fields#97.
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Category | ⇒ | Fields |
This works as described.
Just one comment (besides brian's comment).
Before patch we have:
banners
headers
sampledata
After patch i get (notice the slashes):
\banners
\headers
\sampledata
\sampledata\fruitshop
\sampledata\parks
\sampledata\parks\animals
\sampledata\parks\landscape
So, wouldn't it be more consistent with other tree like views in select boxes (ex: menu items) if we have?:
banners
headers
sampledata
- fruitshop
- parks
- - animals
- - landscape
Not sure as this time it refers to a real physical tree not a virtual one.
hum, but also remeber we will get huge values if we put the whole tree
excellent point
On 5 June 2016 at 13:16, andrepereiradasilva notifications@github.com
wrote:
hum, but also remeber we will get huge values if we put the whole tree
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10722 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8Rs2mNpLlu5efZIygUZv0nEvnuKlks5qIr4YgaJpZM4IuGx4
.
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
This PR has received new commits.
CC: @brianteeman
The tree is describing a real path in the Filesystem, so I would show it like that.
I put in the root and the tree was big, but the page got still loaded decently.
It's true that it may become more readable in some cases (in others probably not).
However the code change becomes much more complex. Currently, we just use the API and return what we get.
I would leave this PR with the full path like it currently is, and if we want we can change it to a tree later.
ok for me
Can somebody else test it please?
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
Milestone |
Added: |
Thanks!
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-07 19:42:35 |
Closed_By | ⇒ | wilsonge |
Labels |
Removed:
?
|
I have tested this item✅ successfully on ad1dfa3
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10722.