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

Update/partial rewrite of editable-grid #756

Merged
merged 22 commits into from
Oct 27, 2023
Merged

Conversation

silasporth
Copy link
Contributor

While working on a project, I came across the editable-grid and wanted to use it. But I found some bugs that I couldn't fix unless I ported the whole structure to my project. Well, I did that because I liked the idea of the base so much and then I decided to fix the bugs here as well.

What I did:

  • Added many Javadoc comments for a better understanding of the project
  • Changed the visibility of most the methods for an increased extensibility
  • Renamed a lot of files and methods to be more appropriate
  • Updated the EditableDataTable to the newest version of Wicket Extensions' DataTable
  • Fixed the sortability of EditableListDataProviders
  • Redesigned the Add Feature with AddRowToolbar
  • Refactored the editable columns
  • Fixed the form validation done by EditableTableSubmitLink
  • Ajax ready Toolbars
  • Expanded the accessibilty of EditableGrid

If there are problems with any of these changes, I'm happy to help.

Copy link
Member

@martin-g martin-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

The API breaks may make someone unhappy though! An upgrade from 9.x to 10.x will be painful!

<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4j.version}</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't that for testing too ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

Copy link
Contributor

@solomax solomax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

with minor changes :)

@silasporth silasporth temporarily deployed to CI_DEPLOY October 27, 2023 05:58 — with GitHub Actions Inactive
@solomax solomax merged commit 174a872 into wicketstuff:master Oct 27, 2023
1 check passed
@solomax
Copy link
Contributor

solomax commented Oct 27, 2023

Thanks for the contribution @silasporth !

@ilgrosso
Copy link
Contributor

Hi @silasporth I am trying to understand how to update my code from 10.0.0-M1 to 10.0.0 (see #849) but having some issues: is there any usage sample around?

@silasporth
Copy link
Contributor Author

Hi @silasporth I am trying to understand how to update my code from 10.0.0-M1 to 10.0.0 (see #849) but having some issues: is there any usage sample around?

Hi, currently there is only the example in editable-grid-parent/editable-grid-examples but it should be working and it shows a minimal example.

@ilgrosso
Copy link
Contributor

Thank you @silasporth I was able to get around it: apache/syncope@f00ad78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants