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

Unable to add new columns after 11th column created using Version: 1.1.3 #31

Closed
jattwood opened this issue Apr 18, 2024 · 9 comments
Closed
Labels
release/1.1.4 Issues and tasks related to version 1.1.4. release/13.0.1 Issues and tasks related to version 13.0.1. type/bug Task is a bug. Fix it please!

Comments

@jattwood
Copy link
Contributor

Very strange an intermittent issue with Version: 1.1.3 on Umbraco 11 that we have discovered with tables greater than 11 cells when the Limbo tables are used in the block grid. I can't confirm if this happens when not in the block grid, but I would image this issue exists there as well. Under certain scenarios, we are unable to add additional columns or delete rows when these conditions exist. No errors outputted to console.

To replicate:

  1. Create a table with 1 row 12 cells.

image

  1. Label each cell with values. I use number 1 through 11, leave the 12th cell with no value.
  2. Click create
  3. Reopen the table block from your grid
  4. Try to create another column, "Add Column" button does not work, delete row does not work

image

Even if you add a value to your 12th cell, you can't add additional values. Also the delete row function does not work during this scenario.

You can bump the table to make it start working again by deleting a column, but then you lose the data in that entire column across the board, so it's not a viable workaround.

@jattwood
Copy link
Contributor Author

Just working with this again. Regardless of what we do, we are unable to build a table greater than 12 columns. To recreate, just try to add new columns to a new table - click add column 11X to replicate. At the 12th column, you are unable to add any more columns.

@jattwood
Copy link
Contributor Author

Can you please update line 28 in TableDataEditor.js to a number larger than 12? Maybe 24?

image

@hfloyd
Copy link

hfloyd commented Jul 11, 2024

I'm bumping up against this issue today. As per a prior conversation... @abjerner noticed the limitation and wasn't sure why it was in place, which leads me to believe it could be removed without any ill effects.

@jattwood
Copy link
Contributor Author

Hey Heather! Yeah I did a PR fix for this. Just waiting on it's integration and release as well! Cheers.

@hfloyd
Copy link

hfloyd commented Jul 11, 2024

Ah, I see your two PRs:
#32
#33

I'm going to try merging your fix in the "v1" branch, since I need it fixed for a v10 site. (I just compile my own custom NuGet package from these fixes since I usually am too impatient to wait for official releases. 😉

@abjerner
Copy link
Member

Sorry for the lack of action. I'm caught up in too many different things, so even small issues like this end up not being fixed/merged for waaaay too long. I'll try to find some time looking into this soon 😉

@abjerner
Copy link
Member

I've now pushed new releases for both Umbraco 10 and 13:

@abjerner abjerner added type/bug Task is a bug. Fix it please! release/1.1.4 Issues and tasks related to version 1.1.4. release/13.0.1 Issues and tasks related to version 13.0.1. labels Jul 11, 2024
@jattwood
Copy link
Contributor Author

Awesome thanks. Heather - for Sprott lol...

@jattwood
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/1.1.4 Issues and tasks related to version 1.1.4. release/13.0.1 Issues and tasks related to version 13.0.1. type/bug Task is a bug. Fix it please!
Projects
None yet
Development

No branches or pull requests

3 participants