Project

General

Profile

Actions

解析Bug #292

closed
XF ZY

Toggle checkbox should only on checkbox zone not whole line

解析Bug #292: Toggle checkbox should only on checkbox zone not whole line

Added by Xihua Fan over 2 years ago. Updated over 1 year ago.

Status:
已关闭
Priority:
Assignee:
Target version:
IT: Easytryck (Sweden) - v3.0
Start date:
08/21/2023
Due date:
08/21/2023
% Done:

100%

Estimated time:
8.00 h
Spent time:

Description

Task Details
  1. ON/OFF 复选框的时候,只能点击复选框才有效,现在附近(行区域)都可以ON/OFF,可能产生误操作
Input
  • Tom & Jennifer
Output
  • Report and Solution

Others


Files

checkbox.png (284 KB) checkbox.png Xihua Fan, 07/19/2023 03:13 AM
checkbox.png

XF Updated by Xihua Fan over 2 years ago Actions #1

  • Copied from 修改功能 #291: New design for print style (some need to be highlighted) added

XF Updated by Xihua Fan over 2 years ago Actions #2

  • 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 进行中
checkbox.png

XF Updated by Xihua Fan over 2 years ago Actions #3

  • Description updated (diff)

XF Updated by Xihua Fan over 2 years ago Actions #4

  • Tracker changed from 修改功能 to 解析Bug

XF Updated by Xihua Fan over 2 years ago Actions #5

  • Due date changed from 07/21/2023 to 07/28/2023
  • Start date changed from 07/20/2023 to 07/24/2023

ZY Updated by Zhongbao Ye over 2 years ago Actions #6

  • Due date changed from 07/28/2023 to 08/21/2023
  • Start date changed from 07/24/2023 to 08/21/2023

ZY Updated by Zhongbao Ye over 2 years ago Actions #7

  • % Done changed from 0 to 60

Output(2023/08/22) 居家

  • 目前实现点击复选框,可以勾选当前行,再次点击复选框,取消勾选。点击列表的其他地方,复选框不会受影响。
  • 测试过程中发现,列表中显示已勾选,但是联动到弹窗中时,获取订单列表已勾选的内容时,部分已勾选的行,获取不到。还需要调查原因。

ZY Updated by Zhongbao Ye over 2 years ago Actions #8

  • 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
解决方案分析: 需要改造这3个核心方法:_selectRow(),_unselectRow(),_cleanSelectRows()
  • 方案一:
    清空如下方法体的内容部分,保留方法体
    1. _selectRow: function(c){}
    2. _unselectRow: function(c){}
    3. _cleanSelectRows: function(){}
  • 方案二:
    找到调用该3个方法的地方(全局有12处),从源头删除方法调用,并删除该3个方法。
结论:
  • 采用方案一 :
    优点:代码改动小,可把控性强,风险更小。
    缺点:代码有冗余。
  • 方案二:
    优点:代码更整洁,没有冗余代码。
    缺点:改造范围大,不可控因素多,风险大。

XF Updated by Xihua Fan over 2 years ago Actions #9

  • Copied from deleted (修改功能 #291: New design for print style (some need to be highlighted))

XF Updated by Xihua Fan over 1 year ago Actions #10

  • Target version set to v3.0
Actions

Also available in: PDF Atom