解析Bug #292
Toggle checkbox should only on checkbox zone not whole line
Description
Task Details
- ON/OFF 复选框的时候,只能点击复选框才有效,现在附近(行区域)都可以ON/OFF,可能产生误操作
- Tom & Jennifer
- Report and Solution
Others
Files
Updated by Xihua Fan over 2 years ago
- Copied from 修改功能 #291: New design for print style (some need to be highlighted) added
Updated by Xihua Fan over 2 years ago
- File checkbox.png checkbox.png added
- Subject changed from Toggle checkbox should only to Toggle checkbox should only on checkbox zone not whole line
- Status changed from 新建 to 进行中
Updated by Xihua Fan over 2 years ago
- Due date changed from 07/21/2023 to 07/28/2023
- Start date changed from 07/20/2023 to 07/24/2023
Updated by Zhongbao Ye over 2 years ago
- Due date changed from 07/28/2023 to 08/21/2023
- Start date changed from 07/24/2023 to 08/21/2023
Updated by Zhongbao Ye over 2 years ago
- % Done changed from 0 to 60
Output(2023/08/22) 居家
- 目前实现点击复选框,可以勾选当前行,再次点击复选框,取消勾选。点击列表的其他地方,复选框不会受影响。
- 测试过程中发现,列表中显示已勾选,但是联动到弹窗中时,获取订单列表已勾选的内容时,部分已勾选的行,获取不到。还需要调查原因。
Updated by Zhongbao Ye over 2 years ago
- Status changed from 进行中 to 已关闭
- % Done changed from 60 to 100
Output(2023/08/23 居家)
- Solution: Operations on list rows require these actions(selectRow/unselectRow/cleanSelectRows) to be removed.
ui\desktop\js\order.plugin.js
protected\views\layouts\desktop.php
- 方案一:
清空如下方法体的内容部分,保留方法体
1. _selectRow: function(c){}
2. _unselectRow: function(c){}
3. _cleanSelectRows: function(){} - 方案二:
找到调用该3个方法的地方(全局有12处),从源头删除方法调用,并删除该3个方法。
- 采用方案一 :
优点:代码改动小,可把控性强,风险更小。
缺点:代码有冗余。 - 方案二:
优点:代码更整洁,没有冗余代码。
缺点:改造范围大,不可控因素多,风险大。
Updated by Xihua Fan over 2 years ago
- Copied from deleted (修改功能 #291: New design for print style (some need to be highlighted))