Uploaded image for project: 'ZK Spreadsheet'
  1. ZK Spreadsheet
  2. ZSS-692

Label may be mismatched in ClearCellHandler class

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: 3.5.0 RC
    • Fix Version/s: 3.5.0
    • Component/s: None
    • Security Level: Jimmy
    • Labels:
      None

      Description

      Please refer to lines 75-86 in org.zkoss.zss.ui.impl.ua.ClearCellHandler class.

      It appears that the label is reversed for ALL and CONTENT.

        Activity

        neillee neillee created issue -
        Hide
        henrichen henrichen added a comment - - edited
        		String label = null;
        		switch (_type) {
        		case ALL:
        			label = Labels.getLabel("zss.undo.clearContents");
        			break;
        		case CONTENT:
        			label = Labels.getLabel("zss.undo.clearAll");
        			break;
        		case STYLE:
        			label = Labels.getLabel("zss.undo.clearStyles");
        			break;
        		}
        
        Show
        henrichen henrichen added a comment - - edited String label = null ; switch (_type) { case ALL: label = Labels.getLabel( "zss.undo.clearContents" ); break ; case CONTENT: label = Labels.getLabel( "zss.undo.clearAll" ); break ; case STYLE: label = Labels.getLabel( "zss.undo.clearStyles" ); break ; }
        Show
        henrichen henrichen added a comment - 3.5 https://github.com/zkoss/zkspreadsheet/commit/6565758e4ea665569cd1e84172900e83a493c932
        Hide
        henrichen henrichen added a comment -

        Fixed.

        Show
        henrichen henrichen added a comment - Fixed.
        henrichen henrichen made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 3.5.0 [ 12609 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Jenkins Jenkins
            Reporter:
            neillee neillee
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: