-
Notifications
You must be signed in to change notification settings - Fork 2
UFAL/Edit bitstreams table UI fix #1131
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
base: dtq-dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes the visible synchronization status column from the bitstreams/bundles edit item view table by removing the corresponding table cell that displayed synchronization information and checksums.
Key Changes
- Removed the synchronization status/checksum column (
columnSizes.columns[4]) from the bitstream table rows - Cleaned up the table structure to eliminate the whitespace caused by the removed column
- The
#checksumtemplate andisBitstreamSynchronized()method remain in use elsewhere in the component
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@jr-rk Wait, do we want to remove this synchronization column? Won't it break synchonization info? |
milanmajchrak
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check the comment
…display of bundle controls in correct column
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...em-page/item-bitstreams/item-edit-bitstream-bundle/item-edit-bitstream-bundle.component.html
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...em-page/item-bitstreams/item-edit-bitstream-bundle/item-edit-bitstream-bundle.component.scss
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Problem description
Cherry-pick from #992
Removed visible whitespace column from bitstreams/bundles edit item view by removing remaining sync. info table cells.
Copilot review