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

pkp/pkp-lib#9744 Rename the "TableNext" component to "Table" and migrate native tables with class "pkpTable" #407

Merged
merged 7 commits into from
Sep 12, 2024

Conversation

blesildaramirez
Copy link
Collaborator

@blesildaramirez blesildaramirez commented Sep 5, 2024

As part of this cleanup, we also migrated some native tables that rely on the styling from the "pkpTable" class. We decided it's best to migrate these tables to use the new table component as well.
These are the following pages/tables:

  1. Administration -> System Information: Version History table
  2. Administration -> System Information: Server Information table
  3. Administration -> System Information: OJS/OMP/OPS Configuration
  4. As a reviewer, view a submission page and click "View All Submission Details"
  5. Statistics -> Users page: Registered Users table
  6. Submissions -> View Submission -> Publication -> Contributors -> Click Preview: List of Contributors table
  7. DOIs -> Click question mark icon (?): DOI Statuses table
  8. Statistics -> Publications (Articles)-> Click Download Report: Report parameters table
  9. Statistics -> Issues -> Click Download Report: Report parameters table
  10. Statistics -> Context (Journal)-> Click Download Report: Report parameters table
  11. ⚠️ URNPubIdPlugin.php - this one hasn't been migrated yet, will address on a separate PR soon.

@blesildaramirez blesildaramirez changed the title pkp/pkp-lib#9744 Rename the "TableNext" component to "Table" pkp/pkp-lib#9744 Rename the "TableNext" component to "Table" and migrate native tables with class "pkpTable" Sep 10, 2024
@blesildaramirez blesildaramirez merged commit 7cd7278 into pkp:main Sep 12, 2024
4 of 5 checks passed
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.

2 participants