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

Edit record not possible #527

Closed
rjahn opened this issue Dec 6, 2024 · 11 comments
Closed

Edit record not possible #527

rjahn opened this issue Dec 6, 2024 · 11 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@rjahn
Copy link
Member

rjahn commented Dec 6, 2024

Screen: Contacts (DB)

Press the pencil beside the table:

image

The response contains the comman property (startEditing):

        "changedComponents": [
            {
                "id": "T624",
                "startEditing": true
            }
        ],

which should switch the table in edit mode for the current selected record/column.

fyi: A command property is a dynamic property only, for only one response.

@rjahn rjahn added the bug Something isn't working label Dec 6, 2024
@rjahn rjahn added this to the 2.3 milestone Dec 6, 2024
gimmixAT added a commit that referenced this issue Dec 9, 2024
@gimmixAT
Copy link
Contributor

gimmixAT commented Dec 9, 2024

this should work correctly again

@rjahn
Copy link
Member Author

rjahn commented Dec 10, 2024

Edit action is working

BUT:

  • cell-height is growing 1px
  • In Lazy-Loading screen: edit and Escape (cancel) lets ACTIVE cell flickr

@gimmixAT
Copy link
Contributor

The cell / row height issue should be fixed.
I can't repdroduce the flicker in FF or Chrome on WIndows. I guess you are seeing this on macOS @rjahn ?

@rjahn
Copy link
Member Author

rjahn commented Dec 13, 2024

Flickr is with macOS and Chrome, but only if developer tools are embedded and only if you press the pencil - not if you double click with mouse.

@gimmixAT
Copy link
Contributor

@rjahn could you please attach a short screencap, I am not able to reproduce any flicker in chrome on macOS

@rjahn
Copy link
Member Author

rjahn commented Dec 14, 2024

Sure:

flickering.mp4

The difference is that button click sends a request and in case of developer console, all images are requested again?

I saw that images are requested for change row selection? Why?

Without developer tools, the column doesn't flickr.

@gimmixAT
Copy link
Contributor

gimmixAT commented Dec 16, 2024

In your case the caching is disabled in the developer mode. So if an image (the checkbox) is rerendered the image isn't fetched from the cache and reloaded hence the flicker.

@rjahn
Copy link
Member Author

rjahn commented Dec 16, 2024

You're right!

The only open question here: Why does caching work in case of double-click a cell and pressing Esc vs. Pencil and Esc?

@gimmixAT
Copy link
Contributor

gimmixAT commented Dec 16, 2024

doubleclicking it only starts the editing session on the client apparently since there seems to be no request sent at that moment. So just the cell rerenders to display the editor.

When clicking the pen, the editing session is triggered by the server response, which in turn causes a rerender of the table content

@rjahn
Copy link
Member Author

rjahn commented Dec 17, 2024

From my point of view, there should be no difference if user double clicks or gets the response which will start editing?
At the end, the editor starts editing? Do we have two different edit modes?

Are we safe here?

@gimmixAT
Copy link
Contributor

gimmixAT commented Dec 17, 2024

The end result is basically the same so we should be safe on that end. Just how react propagates those changes of the editing mode is different and causes some rendering side effects (like unnecessary DOM updates) on the way.
This should be remediable once we retool the react stack a bit and optimize more for recycling components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants