Skip to content

Migrate ExportRecordTable from BTable to GTable#21727

Merged
davelopez merged 2 commits intogalaxyproject:devfrom
itisAliRH:btable-to-gtable-export-record-table
Feb 4, 2026
Merged

Migrate ExportRecordTable from BTable to GTable#21727
davelopez merged 2 commits intogalaxyproject:devfrom
itisAliRH:btable-to-gtable-export-record-table

Conversation

@itisAliRH
Copy link
Copy Markdown
Member

This PR migrates ExportRecordTable.vue from Bootstrap-Vue's BTable to our custom GTable component, as part of the ongoing effort tracked in #21703.

Before After
image image

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

- Add separate text alignment for header labels (text-center, text-left, text-right)
- Keep sort icons always aligned to the right using flexbox
- Fix body cell alignment to use standard text-center classes
- When align=center, only the label is centered while icon stays right
- Replace BTable with GTable component
- Add TableField type import for proper typing
- Update field definitions with align property (center for status columns)
- Fix Date object TypeScript errors by converting to strings with toLocaleString()
- Maintain full backward compatibility with existing slot syntax
- No functional changes, pure migration

ref galaxyproject#21703 (1/48 components)
@github-actions github-actions Bot added this to the 26.1 milestone Feb 3, 2026
@itisAliRH itisAliRH added the kind/refactoring cleanup or refactoring of existing code, no functional changes label Feb 3, 2026
Copy link
Copy Markdown
Contributor

@davelopez davelopez left a comment

Choose a reason for hiding this comment

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

Looks pretty good! Thank you!

@davelopez davelopez merged commit a7c9eda into galaxyproject:dev Feb 4, 2026
41 of 42 checks passed
@itisAliRH itisAliRH deleted the btable-to-gtable-export-record-table branch February 4, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/UI-UX kind/refactoring cleanup or refactoring of existing code, no functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants