From 7ca1858fcbec14f37894e2439f95a9edb3eea754 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jindra=20Pet=C5=99=C3=ADk?= Date: Wed, 14 May 2025 00:23:51 +0200 Subject: [PATCH] Simple editor - editing convolution filter --- CHANGELOG.md | 2 +- .../flash/types/filters/ConvolveOp.java | 4 +- .../flash/types/filters/Filtering.java | 2 +- .../flash/easygui/FiltersTreeTable.java | 45 +- .../properties/ConvolutionMatrixEditor.java | 431 ++++++++++++++++++ .../gui/generictageditors/ColorEditor.java | 5 +- 6 files changed, 454 insertions(+), 35 deletions(-) create mode 100644 src/com/jpexs/decompiler/flash/easygui/properties/ConvolutionMatrixEditor.java diff --git a/CHANGELOG.md b/CHANGELOG.md index 5006c286c..ec727e439 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -22,7 +22,7 @@ All notable changes to this project will be documented in this file. - Option to enter custom zoom level by clicking on zoom percentage label - Show in Simple editor context menu item for timelined items (sprites, buttons, swfs) - Simple editor - change background color -- Simple editor - filters (without editing convolution) +- Simple editor - filters ### Fixed - [#2424] DefineEditText handling of letterSpacing, font size on incorrect values diff --git a/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/ConvolveOp.java b/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/ConvolveOp.java index dd67661a9..e2b525902 100644 --- a/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/ConvolveOp.java +++ b/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/ConvolveOp.java @@ -142,8 +142,8 @@ public class ConvolveOp implements BufferedImageOp, RasterOp { float[] kvals = kernel.getKernelData(null); - for (int x = srcX - left; x < srcX + srcWidth + left + 1; x++) { - for (int y = srcY - top; y < srcY + srcHeight + top + 1; y++) { + for (int x = srcX - left; x < srcX + srcWidth + left; x++) { + for (int y = srcY - top; y < srcY + srcHeight + top; y++) { float a; if (preserveAlpha) { boolean outSide = false; diff --git a/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/Filtering.java b/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/Filtering.java index 47ce0e554..d4909c15a 100644 --- a/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/Filtering.java +++ b/libsrc/ffdec_lib/src/com/jpexs/decompiler/flash/types/filters/Filtering.java @@ -669,7 +669,7 @@ public class Filtering { int srcHeight ) { Kernel kernel = new Kernel(w, h, matrix); - BufferedImage dst = new BufferedImage(src.getWidth() + 1, src.getHeight() + 1, src.getType()); + BufferedImage dst = new BufferedImage(src.getWidth(), src.getHeight(), src.getType()); BufferedImageOp op = new ConvolveOp( kernel, new RenderingHints(null), diff --git a/src/com/jpexs/decompiler/flash/easygui/FiltersTreeTable.java b/src/com/jpexs/decompiler/flash/easygui/FiltersTreeTable.java index 4c3d38ed5..f7f36c505 100644 --- a/src/com/jpexs/decompiler/flash/easygui/FiltersTreeTable.java +++ b/src/com/jpexs/decompiler/flash/easygui/FiltersTreeTable.java @@ -16,6 +16,7 @@ */ package com.jpexs.decompiler.flash.easygui; +import com.jpexs.decompiler.flash.easygui.properties.ConvolutionMatrixEditor; import com.jpexs.decompiler.flash.easygui.properties.GradientEditor; import com.jpexs.decompiler.flash.easygui.properties.PropertyEditor; import com.jpexs.decompiler.flash.ecma.EcmaNumberToString; @@ -23,10 +24,8 @@ import com.jpexs.decompiler.flash.ecma.EcmaScript; import com.jpexs.decompiler.flash.gui.AppStrings; import com.jpexs.decompiler.flash.gui.View; import com.jpexs.decompiler.flash.gui.generictageditors.BooleanEditor; -import com.jpexs.decompiler.flash.gui.generictageditors.ChangeListener; import com.jpexs.decompiler.flash.gui.generictageditors.ColorEditor; import com.jpexs.decompiler.flash.gui.generictageditors.FloatEditor; -import com.jpexs.decompiler.flash.gui.generictageditors.GenericTagEditor; import com.jpexs.decompiler.flash.gui.generictageditors.NumberEditor; import com.jpexs.decompiler.flash.gui.generictageditors.ValueNormalizer; import com.jpexs.decompiler.flash.types.BasicType; @@ -35,6 +34,7 @@ import com.jpexs.decompiler.flash.types.RGBA; import com.jpexs.decompiler.flash.types.annotations.Reserved; import com.jpexs.decompiler.flash.types.annotations.SWFType; import com.jpexs.decompiler.flash.types.filters.COLORMATRIXFILTER; +import com.jpexs.decompiler.flash.types.filters.CONVOLUTIONFILTER; import com.jpexs.decompiler.flash.types.filters.ColorMatrixConvertor; import com.jpexs.decompiler.flash.types.filters.FILTER; import de.javagl.treetable.JTreeTable; @@ -84,9 +84,7 @@ import javax.swing.table.DefaultTableCellRenderer; import javax.swing.table.TableCellEditor; import javax.swing.tree.DefaultMutableTreeNode; import javax.swing.tree.DefaultTreeCellRenderer; -import javax.swing.tree.DefaultTreeModel; import javax.swing.tree.TreeModel; -import javax.swing.tree.TreeNode; import javax.swing.tree.TreePath; /** @@ -535,6 +533,8 @@ public class FiltersTreeTable extends JTreeTable { return 0; } }; + } else if ("matrix".equals(filterField.field.getName()) && filterField.filter instanceof CONVOLUTIONFILTER) { + editor = new ConvolutionMatrixEditor((CONVOLUTIONFILTER) filterField.filter); } else if ("gradientColors".equals(filterField.field.getName())) { editor = new GradientEditor(filterField.filter); } else if (realValue.getClass() == Boolean.class) { @@ -710,7 +710,11 @@ public class FiltersTreeTable extends JTreeTable { } else { label.setText(value.toString() + units); } - if (fieldValue != null) { + if (fieldValue != null) { + if ("matrix".equals(filterValue.filterField.field.getName()) && filterValue.filterField.filter instanceof CONVOLUTIONFILTER) { + component = new ConvolutionMatrixEditor((CONVOLUTIONFILTER) filterValue.filterField.filter); + } + if ("gradientColors".equals(filterValue.filterField.field.getName())) { component = new GradientEditor(filterValue.filterField.filter); } @@ -724,28 +728,9 @@ public class FiltersTreeTable extends JTreeTable { panel.setOpaque(false); component = panel; } - if (fieldValue.getClass() == RGBA.class) { - JPanel panel = new JPanel(new BorderLayout()); - JButton buttonChange = new JButton("") { - - @Override - protected void paintComponent(Graphics g) { - g.setColor(getBackground()); - g.fillRect(0, 0, getWidth(), getHeight()); - super.paintBorder(g); - } - - }; - buttonChange.setToolTipText(AppStrings.translate("button.selectcolor.hint")); - buttonChange.setCursor(new Cursor(Cursor.HAND_CURSOR)); - buttonChange.setBorderPainted(true); - buttonChange.setBorder(BorderFactory.createBevelBorder(BevelBorder.RAISED)); - Dimension colorDim = new Dimension(16, 16); - buttonChange.setSize(colorDim); - buttonChange.setPreferredSize(colorDim); - buttonChange.setBackground(((RGBA) fieldValue).toColor()); - panel.add(buttonChange, BorderLayout.WEST); - component = panel; + if (fieldValue.getClass() == RGBA.class) { + component = new ColorEditor(filterValue.filterField.toString(), filterValue.filterField.filter, filterValue.filterField.field, -1, RGBA.class); + component.setToolTipText(AppStrings.translate("button.selectcolor.hint")); } } } @@ -967,6 +952,12 @@ public class FiltersTreeTable extends JTreeTable { if ("gradientRatio".equals(field.getName())) { continue; } + if (filter instanceof CONVOLUTIONFILTER) { + if ("matrixX".equals(field.getName()) + || "matrixY".equals(field.getName())) { + continue; + } + } if (filter instanceof COLORMATRIXFILTER) { if ("matrix".equals(field.getName())) { diff --git a/src/com/jpexs/decompiler/flash/easygui/properties/ConvolutionMatrixEditor.java b/src/com/jpexs/decompiler/flash/easygui/properties/ConvolutionMatrixEditor.java new file mode 100644 index 000000000..15f9ee5cf --- /dev/null +++ b/src/com/jpexs/decompiler/flash/easygui/properties/ConvolutionMatrixEditor.java @@ -0,0 +1,431 @@ +/* + * Copyright (C) 2025 JPEXS + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ +package com.jpexs.decompiler.flash.easygui.properties; + +import com.jpexs.decompiler.flash.ecma.EcmaScript; +import com.jpexs.decompiler.flash.gui.View; +import com.jpexs.decompiler.flash.gui.generictageditors.ChangeListener; +import com.jpexs.decompiler.flash.types.filters.CONVOLUTIONFILTER; +import java.awt.BorderLayout; +import java.awt.Color; +import java.awt.Component; +import java.awt.Dimension; +import java.awt.FlowLayout; +import java.awt.GridBagConstraints; +import java.awt.GridBagLayout; +import java.awt.Point; +import java.awt.Window; +import java.awt.event.ActionEvent; +import java.awt.event.MouseAdapter; +import java.awt.event.MouseEvent; +import java.awt.event.WindowAdapter; +import java.awt.event.WindowEvent; +import java.util.ArrayList; +import java.util.EventObject; +import java.util.List; +import javax.swing.BorderFactory; +import javax.swing.Box; +import javax.swing.BoxLayout; +import javax.swing.JButton; +import javax.swing.JDialog; +import javax.swing.JLabel; +import javax.swing.JPanel; +import javax.swing.JRootPane; +import javax.swing.JTable; +import javax.swing.JTextField; +import javax.swing.SwingUtilities; +import javax.swing.event.CellEditorListener; +import javax.swing.event.ChangeEvent; +import javax.swing.plaf.basic.BasicTableUI; +import javax.swing.table.DefaultTableCellRenderer; +import javax.swing.table.DefaultTableModel; +import javax.swing.table.TableCellEditor; + +/** + * + * @author JPEXS + */ +public class ConvolutionMatrixEditor extends JPanel implements PropertyEditor { + + private final JLabel previewLabel; + private final CONVOLUTIONFILTER filter; + private List changeListeners = new ArrayList<>(); + + private void updateLabel() { + StringBuilder sb = new StringBuilder(); + sb.append(filter.matrixX); + sb.append("x"); + sb.append(filter.matrixY); + sb.append(" ["); + int length = filter.matrix.length; + int i = 0; + for (int y = 0; y < filter.matrixY; y++) { + if (y > 0) { + sb.append("; "); + } + for (int x = 0; x < filter.matrixX; x++) { + if (x > 0) { + sb.append(", "); + } + sb.append(EcmaScript.toString(filter.matrix[i])); + i++; + } + } + sb.append("]"); + previewLabel.setText(sb.toString()); + } + + public ConvolutionMatrixEditor(CONVOLUTIONFILTER filter) { + this.filter = filter; + + setLayout(new BorderLayout()); + + previewLabel = new JLabel(); + + updateLabel(); + + previewLabel.addMouseListener(new MouseAdapter() { + @Override + public void mousePressed(MouseEvent e) { + if (SwingUtilities.isLeftMouseButton(e)) { + JDialog dialog = new JDialog(); + dialog.setUndecorated(true); + dialog.setResizable(false); + dialog.getRootPane().setWindowDecorationStyle(JRootPane.NONE); + MatrixEditorPanel matrixEditorPanel = new MatrixEditorPanel(dialog); + dialog.setContentPane(matrixEditorPanel); + dialog.pack(); + + Window window = SwingUtilities.getWindowAncestor(ConvolutionMatrixEditor.this); + + dialog.setLocationRelativeTo(window); + Point loc = SwingUtilities.convertPoint(previewLabel, 0, 0, window); + if (loc.x + dialog.getWidth() > window.getWidth()) { + loc.x -= loc.x + dialog.getWidth() - window.getWidth(); + } + if (loc.y + dialog.getHeight() > window.getHeight()) { + loc.y -= loc.y + dialog.getHeight() - window.getHeight(); + } + SwingUtilities.convertPointToScreen(loc, window); + + dialog.setLocation(loc); + dialog.addWindowListener(new WindowAdapter() { + @Override + public void windowDeactivated(WindowEvent e) { + filter.matrixX = matrixEditorPanel.getMatrixX(); + filter.matrixY = matrixEditorPanel.getMatrixY(); + filter.matrix = matrixEditorPanel.getMatrix(); + updateLabel(); + dialog.setVisible(false); + for (ChangeListener l : changeListeners) { + l.change(ConvolutionMatrixEditor.this); + } + } + }); + dialog.setVisible(true); + } + } + }); + + add(previewLabel, BorderLayout.WEST); + + } + + private class MatrixEditorPanel extends JPanel { + + private final JTable matrixTable; + private final JDialog dialog; + + private final JButton enlargeVerticalButton; + private final JButton shrinkVerticalButton; + private final JButton enlargeHorizontalButton; + private final JButton shrinkHorizontalButton; + + public int getMatrixX() { + return matrixTable.getModel().getColumnCount(); + } + + public int getMatrixY() { + return matrixTable.getModel().getRowCount(); + } + + public float[] getMatrix() { + int cols = getMatrixX(); + int rows = getMatrixY(); + float[] matrix = new float[cols * rows]; + for (int y = 0; y < rows; y++) { + for (int x = 0; x < cols; x++) { + matrix[y * cols + x] = (float) matrixTable.getModel().getValueAt(y, x); + } + } + return matrix; + } + + public MatrixEditorPanel(JDialog dialog) { + setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5)); + DefaultTableModel model = new DefaultTableModel(filter.matrixY, filter.matrixX) { + @Override + public Class getColumnClass(int columnIndex) { + return float.class; + } + }; + for (int y = 0; y < filter.matrixY; y++) { + for (int x = 0; x < filter.matrixX; x++) { + model.setValueAt(filter.matrix[y * filter.matrixX + x], y, x); + } + } + + matrixTable = new JTable(model); + + matrixTable.setDefaultRenderer(float.class, new DefaultTableCellRenderer() { + @Override + public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { + JLabel label = (JLabel) super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column); + label.setText(EcmaScript.toString(value)); + return label; + } + }); + + matrixTable.setDefaultEditor(float.class, new TableCellEditor() { + + private List listeners = new ArrayList<>(); + private JTextField textField = new JTextField(); + + @Override + public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column) { + textField.setText(EcmaScript.toString(value)); + return textField; + } + + @Override + public Object getCellEditorValue() { + String input = textField.getText(); + try { + return Float.valueOf(input); + } catch (NumberFormatException nfe) { + return 0.0f; + } + } + + @Override + public boolean isCellEditable(EventObject anEvent) { + return true; + } + + @Override + public boolean shouldSelectCell(EventObject anEvent) { + return true; + } + + @Override + public boolean stopCellEditing() { + List listeners2 = new ArrayList<>(listeners); + for (CellEditorListener l : listeners2) { + l.editingStopped(new ChangeEvent(this)); + } + return true; + } + + @Override + public void cancelCellEditing() { + List listeners2 = new ArrayList<>(listeners); + for (CellEditorListener l : listeners2) { + l.editingCanceled(new ChangeEvent(this)); + } + } + + @Override + public void addCellEditorListener(CellEditorListener l) { + listeners.add(l); + } + + @Override + public void removeCellEditorListener(CellEditorListener l) { + listeners.remove(l); + } + + }); + + matrixTable.setTableHeader(null); + + matrixTable.setUI(new BasicTableUI()); + + if (View.isOceanic()) { + matrixTable.setBackground(Color.WHITE); + } + + setLayout(new GridBagLayout()); + GridBagConstraints gbc = new GridBagConstraints(); + + JPanel buttonsVerticalPanel = new JPanel(new FlowLayout()); + enlargeVerticalButton = new JButton("+"); + enlargeVerticalButton.addActionListener(this::enlargeVerticalActionPerformed); + enlargeVerticalButton.setPreferredSize(new Dimension(30, 30)); + enlargeVerticalButton.setMinimumSize(new Dimension(30, 30)); + shrinkVerticalButton = new JButton("-"); + shrinkVerticalButton.addActionListener(this::shrinkVerticalActionPerformed); + shrinkVerticalButton.setPreferredSize(new Dimension(30, 30)); + shrinkVerticalButton.setMinimumSize(new Dimension(30, 30)); + buttonsVerticalPanel.add(enlargeVerticalButton); + buttonsVerticalPanel.add(shrinkVerticalButton); + + JPanel buttonsHorizontalPanel = new JPanel(); + buttonsHorizontalPanel.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5)); + buttonsHorizontalPanel.setLayout(new BoxLayout(buttonsHorizontalPanel, BoxLayout.Y_AXIS)); + enlargeHorizontalButton = new JButton("+"); + enlargeHorizontalButton.addActionListener(this::enlargeHorizontalActionPerformed); + enlargeHorizontalButton.setPreferredSize(new Dimension(30, 30)); + enlargeHorizontalButton.setMinimumSize(new Dimension(30, 30)); + shrinkHorizontalButton = new JButton("-"); + shrinkHorizontalButton.addActionListener(this::shrinkHorizontalActionPerformed); + shrinkHorizontalButton.setPreferredSize(new Dimension(30, 30)); + shrinkHorizontalButton.setMinimumSize(new Dimension(30, 30)); + buttonsHorizontalPanel.add(Box.createVerticalGlue()); + buttonsHorizontalPanel.add(enlargeHorizontalButton); + buttonsHorizontalPanel.add(shrinkHorizontalButton); + buttonsHorizontalPanel.add(Box.createVerticalGlue()); + + gbc.anchor = GridBagConstraints.CENTER; + gbc.gridx = 0; + gbc.gridy = 0; + + add(matrixTable, gbc); + gbc.gridx++; + add(buttonsHorizontalPanel, gbc); + gbc.gridy = 1; + gbc.gridx = 0; + add(buttonsVerticalPanel, gbc); + this.dialog = dialog; + } + + private void updateDialogPosition() { + Window window = SwingUtilities.getWindowAncestor(ConvolutionMatrixEditor.this); + dialog.setLocationRelativeTo(window); + Point loc = SwingUtilities.convertPoint(previewLabel, 0, 0, window); + if (loc.x + dialog.getWidth() > window.getWidth()) { + loc.x -= loc.x + dialog.getWidth() - window.getWidth(); + } + if (loc.y + dialog.getHeight() > window.getHeight()) { + loc.y -= loc.y + dialog.getHeight() - window.getHeight(); + } + SwingUtilities.convertPointToScreen(loc, window); + dialog.setLocation(loc); + } + + private void enlargeVerticalActionPerformed(ActionEvent aev) { + DefaultTableModel model = (DefaultTableModel) matrixTable.getModel(); + int columnCount = model.getColumnCount(); + int rowCount = model.getRowCount(); + model.setRowCount(rowCount + 2); + + for (int i = 0; i < columnCount; i++) { + model.setValueAt(0f, rowCount, i); + model.setValueAt(0f, rowCount + 1, i); + } + + for (int y = rowCount - 1; y >= 0; y--) { + for (int x = columnCount - 1; x >= 0; x--) { + model.setValueAt(model.getValueAt(y, x), y + 1, x); + } + } + for (int i = 0; i < columnCount; i++) { + model.setValueAt(0f, 0, i); + } + dialog.pack(); + updateDialogPosition(); + shrinkVerticalButton.setEnabled(true); + } + + private void shrinkVerticalActionPerformed(ActionEvent aev) { + DefaultTableModel model = (DefaultTableModel) matrixTable.getModel(); + + int columnCount = model.getColumnCount(); + int rowCount = model.getRowCount(); + + for (int y = 0; y < rowCount - 2; y++) { + for (int x = 0; x < columnCount; x++) { + model.setValueAt(model.getValueAt(y + 1, x), y, x); + } + } + + model.setRowCount(rowCount - 2); + + dialog.pack(); + shrinkVerticalButton.setEnabled(rowCount - 2 > 1); + updateDialogPosition(); + } + + private void enlargeHorizontalActionPerformed(ActionEvent aev) { + DefaultTableModel model = (DefaultTableModel) matrixTable.getModel(); + int columnCount = model.getColumnCount(); + int rowCount = model.getRowCount(); + model.setColumnCount(columnCount + 2); + + for (int i = 0; i < rowCount; i++) { + model.setValueAt(0f, i, columnCount); + model.setValueAt(0f, i, columnCount + 1); + } + + for (int y = rowCount - 1; y >= 0; y--) { + for (int x = columnCount - 1; x >= 0; x--) { + model.setValueAt(model.getValueAt(y, x), y, x + 1); + } + } + for (int i = 0; i < rowCount; i++) { + model.setValueAt(0f, i, 0); + } + dialog.pack(); + updateDialogPosition(); + shrinkHorizontalButton.setEnabled(true); + } + + private void shrinkHorizontalActionPerformed(ActionEvent aev) { + DefaultTableModel model = (DefaultTableModel) matrixTable.getModel(); + + int columnCount = model.getColumnCount(); + int rowCount = model.getRowCount(); + + for (int y = 0; y < rowCount; y++) { + for (int x = 0; x < columnCount - 2; x++) { + model.setValueAt(model.getValueAt(y, x + 1), y, x); + } + } + + model.setColumnCount(columnCount - 2); + + dialog.pack(); + + shrinkHorizontalButton.setEnabled(columnCount - 2 > 1); + updateDialogPosition(); + } + } + + @Override + public boolean save() { + return true; + } + + @Override + public void reset() { + + } + + @Override + public void addChangeListener(ChangeListener l) { + changeListeners.add(l); + } +} diff --git a/src/com/jpexs/decompiler/flash/gui/generictageditors/ColorEditor.java b/src/com/jpexs/decompiler/flash/gui/generictageditors/ColorEditor.java index f0dac0a92..f7e677b30 100644 --- a/src/com/jpexs/decompiler/flash/gui/generictageditors/ColorEditor.java +++ b/src/com/jpexs/decompiler/flash/gui/generictageditors/ColorEditor.java @@ -29,12 +29,9 @@ import java.awt.Color; import java.awt.Component; import java.awt.Cursor; import java.awt.Dimension; -import java.awt.FlowLayout; import java.awt.Graphics; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; -import java.awt.event.FocusAdapter; -import java.awt.event.FocusEvent; import java.lang.reflect.Array; import java.lang.reflect.Field; import java.lang.reflect.Method; @@ -45,7 +42,6 @@ import javax.swing.BorderFactory; import javax.swing.JButton; import javax.swing.JColorChooser; import javax.swing.JDialog; -import javax.swing.JPanel; import javax.swing.JSlider; import javax.swing.JSpinner; import javax.swing.border.BevelBorder; @@ -118,6 +114,7 @@ public class ColorEditor extends FocusablePanel implements GenericTagEditor, Act buttonChange.setSize(colorDim); buttonChange.setPreferredSize(colorDim); add(buttonChange, BorderLayout.WEST); + setBorder(BorderFactory.createEmptyBorder(0, 5, 0, 0)); reset(); }