Feeling Lucky
?
[#7368] - Allows visible text fields within JSortableList
- Closed
- 25 Oct 2015
- Medium
- Build: staging
- Easy Test
- # 7368
- Diff
- sebastienheraud:staging
- Success Human Test Results: 3 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
JSortableList is altering all text fields found in the table.. which is
unnecessary and quite restrictive.
Of course it would be better to use the following selector
".text-area-order" but as there are some locations (ie. com_menus,
com_categories, ..) without the class, and maybe others in 3rd party
components, the easiet way is maybe to search for: [type=text]:hidden
At least, this allows us to have "visible" text fields without
alteration of their value.
Thanks,
Sebastien
Votes
# of Users Experiencing Issue
1/1
Average Importance Score
3.00
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Information Required |
| Labels |
Added:
?
|
||
| Category | ⇒ | JavaScript |
sebastienheraud
- comment
- 7 Jul 2015
Hi Fedik
Sure... here are the instructions:
- Open the following file /administrator/components/com_content/views/articles/tmpl/default.php
- Then add a text input after line 211 < td >< input type="text" id="plop" name="plop" value="x" class="input-mini" / >< /td >
- From the Article Manager, select "Ordering ascending"
- Reorder using the drag & drop grip icon ==> without fix: you'll see that the "x" values are altered... ==> with fix: you'll see that the "x" values are not altered... (after refresh, reorder is still saved = OK) Regards, Sebastien
| Labels |
Removed:
?
|
||
| Status | Information Required | ⇒ | Pending |
Fedik
- comment
- 14 Jul 2015
test ok, works as described
| Easy | No | ⇒ | Yes |
Bloggerschmidt
- comment
- 24 Oct 2015
okay. works as decribed.
teccrow
- comment
- 24 Oct 2015
@test it works.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7368.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 24 Oct 2015
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
rdeutz
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-25 10:11:10 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
sebastienheraud
- comment
- 25 Oct 2015
Thanks for the merge!
Sebastien
| Milestone | |||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone | |||
please add the testing description .... where and how people can test your suggestion, and what expect in result, thanks!