Closed (fixed)
Project:
Commerce Stock
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Oct 2018 at 11:53 UTC
Updated:
9 Feb 2019 at 19:59 UTC
Jump to comment: Most recent
Comments
Comment #2
olafkarsten commentedI'm not sure what the @MrPaulDriver is expecting from us. As long as we don't get other information, I consider this a feature request.
It's not a bug, when we have no integration with another contrib module.
Comment #3
mrpauldriver commentedI don't agree
These labels should not be appearing on each row of a views table. The screen shots are self explanatory in my honest opinion.
Comment #4
olafkarsten commentedI got you. I don't know the internals of Views Entity Form Field, but I guess they expect a certain structure from the widget / likely a standard form element
The StockLevelWidget is somewhat complex and likely don't meet the expectations.
There is an issue to refactor and split the widget in three widgets. That would likely result in a much simpler structure per widget.
#2931754: Split the SimpleStockLevelWidget in three widgets
So it's not only about views integration. The widget should simple follow the docs. Any contrib that expect this, should work then.
I leave this issue open but postponed as a reminder to give @MrPaulDriver the credit he deserves, when we solve the older issue.
Thanks @MrPaulDriver
Comment #5
olafkarsten commentedwe fixed and split the widgets in the mentioned issue. so this one should be solved for any of the three new widgets.