Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Table row selection works incorrectly if clicked inside edit field #3009

Closed
Flaurite opened this issue Aug 24, 2020 · 1 comment
Closed

Table row selection works incorrectly if clicked inside edit field #3009

Flaurite opened this issue Aug 24, 2020 · 1 comment
Assignees
Labels
state: won't fix This will not be worked on type: regression
Milestone

Comments

@Flaurite
Copy link
Contributor

Flaurite commented Aug 24, 2020

Environment

  • Platform version: 7.2
  • Client type: Web

Description of the bug or enhancement

Russian support forum: topic.
Cannot reproduce on 7.1.7 version

  • Download attached project: instancechangetest.zip
  • Edit order.
  • Select first row, edit Price, don't press Enter or move focus.
  • Click inside Price field of another row.
  • ER: selection moves to another row.
  • AR: selection moves to another row and then returns back to the first one.

Together with #3026 it leads also to losing ItemPropertyChangeEvent listener which in turn leads to Sum is not updated when Price is changed after such selection jump.

@Flaurite Flaurite added this to the Release 7.2 milestone Aug 24, 2020
@knstvk knstvk assigned knstvk and unassigned t1-cuba Aug 31, 2020
@knstvk knstvk changed the title Nested collection container does not fire ItemPropertyChangeEvent in some cases Table row selection works incorrectly if clicked inside edit field Sep 3, 2020
@knstvk knstvk assigned Flaurite and unassigned knstvk Sep 3, 2020
@Flaurite
Copy link
Contributor Author

Flaurite commented Sep 4, 2020

This problem the same roots as described here cuba-platform/cuba#2712.

The workaround is: replace non-editable columns by generated column with Label or Table.PlainTextCell:

table.addGeneratedColumn("name", entity -> new Table.PlainTextCell(entity.getName()));

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: won't fix This will not be worked on type: regression
Projects
None yet
Development

No branches or pull requests

3 participants