- πΊπ¦Ukraine ipinchuk
Hi everyone,
I don't understand the benefit of this improvement. But, if it should be, I suggest adding sorting like on the Roles listing page (drag-and-drop). This is more useful than a single weight field.
What do you think, guys?
- πΊπ¦Ukraine kir.lazur@gmail.com Kiev
Hi all,
thanks for contribution into this extension.
I'm agree with @ipinchuk, the full functionality of weight might be achieved with d&d on listing page, not only the field itself. - Status changed to Needs work
over 1 year ago 9:20am 13 May 2023 - Status changed to Needs review
over 1 year ago 8:39am 16 May 2023 - πΊπ¦Ukraine WannaAlwaysBe
Hello everyone,
I also agree with @ipinchuk. Therefore, I removed the weight field from the Email form as it was in my previous patch and extended the ListBuilder with a DraggableListBuilder to provide d&d functionality.
Additionally, I took the hook_update from @murz MR to populate the "weight" with the default value.Please review and provide feedback.
Thanks.
- Status changed to RTBC
over 1 year ago 7:13am 23 May 2023 - πΊπ¦Ukraine ysamoylenko
The changes look good to me. Please commit them.
- Assigned to kir.lazur@gmail.com
-
Kir Lazur β
committed 92529796 on 2.0.x authored by
Murz β
Resolve #3126660 "Weight field"
-
Kir Lazur β
committed 92529796 on 2.0.x authored by
Murz β
- Status changed to Fixed
over 1 year ago 9:04am 29 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.