? ? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
15 Aug 2021

Summary of Changes

Fixing subform styling.
Adopted from joomla 3

Testing Instructions

Apply patch, run node build/build.js --compile-css

Add subform field somewhere, example in mod_custom xml

<field type="subform" name="subform" label="subform" multiple="true">
  <form>
    <field type="text" name="text" label="text" />
    <field type="textarea" name="textarea" label="textarea" />
  </form>
</field>

Actual result BEFORE applying this Pull Request

Screenshot_2021-08-15_14-18-40

Expected result AFTER applying this Pull Request

Screenshot_2021-08-15_14-41-32

Documentation Changes Required

avatar Fedik Fedik - open - 15 Aug 2021
avatar Fedik Fedik - change - 15 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Aug 2021
Category Administration Templates (admin) NPM Change Layout
avatar Fedik
Fedik - comment - 15 Aug 2021

this for 4.0.x

avatar Fedik Fedik - change - 15 Aug 2021
Labels Added: ? NPM Resource Changed
avatar Fedik Fedik - change - 15 Aug 2021
Labels Added: ?
Removed: ?
avatar Fedik Fedik - change - 15 Aug 2021
Title
[4.0] Subform styling for non table layout
[4] Subform styling for non table layout
avatar Fedik Fedik - edited - 15 Aug 2021
avatar Fedik Fedik - change - 15 Aug 2021
Title
[4] Subform styling for non table layout
[4.0] Subform styling for non table layout
avatar Fedik Fedik - edited - 15 Aug 2021
avatar Waleet
Waleet - comment - 16 Aug 2021

Hello Fedik,
I tested your changes successfully on my machine.
Although I noticed that dragging and dropping feels a bit clunky.
You have to be very precise with the mouse to grab an item.
Maybe that is tweakable?
Also while holding on to an item it becomes hard to see what you´re currently holding.
That may be because white on white is hard to see.

Nonetheless good addition :)

avatar Waleet Waleet - test_item - 16 Aug 2021 - Tested successfully
avatar Waleet
Waleet - comment - 16 Aug 2021

I have tested this item successfully on 34a1de2


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

avatar Fedik Fedik - change - 16 Aug 2021
Labels Added: ?
Removed: ?
avatar Fedik
Fedik - comment - 16 Aug 2021

Also while holding on to an item it becomes hard to see what you´re currently holding.

I have added a light background for "draggable" section,
Please test again.

avatar brianteeman
brianteeman - comment - 16 Aug 2021

Can you use the same styling for the dragging as is used when you are ordering a menu item etc. That will make the ui more consistent

avatar Fedik
Fedik - comment - 16 Aug 2021

Can you use the same styling for the dragging as is used when you are ordering a menu item etc. That will make the ui more consistent

yes, if someone tell me where I can copy that ;)

avatar brianteeman
brianteeman - comment - 16 Aug 2021

It's coming from dragula in media/vendor

avatar Fedik
Fedik - comment - 16 Aug 2021

ah, wait, that toxic green? no, not allowed :)

avatar Fedik
Fedik - comment - 16 Aug 2021

okay, changed, however I not like that color.
just for consistency

Screenshot_2021-08-16_17-34-19

avatar AndySDH AndySDH - test_item - 17 Aug 2021 - Tested successfully
avatar AndySDH
AndySDH - comment - 17 Aug 2021

I have tested this item successfully on ff878dd

Thanks, nice UI improvement


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

avatar RickR2H RickR2H - test_item - 17 Aug 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 17 Aug 2021

I have tested this item successfully on ff878dd

This layout is a lot more clear.


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

avatar Quy Quy - change - 17 Aug 2021
The description was changed
Status Pending Ready to Commit
avatar Quy Quy - edited - 17 Aug 2021
avatar Quy
Quy - comment - 17 Aug 2021

RTC


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

avatar Quy Quy - change - 25 Aug 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-08-25 15:28:48
Closed_By Quy
Labels Added: ? ?
Removed: ?
avatar Quy Quy - close - 25 Aug 2021
avatar Quy Quy - merge - 25 Aug 2021
avatar Quy
Quy - comment - 25 Aug 2021

Thanks!

Add a Comment

Login with GitHub to post a comment