UFAL/Edit bitstreams table UI fix - #1131
Conversation
There was a problem hiding this comment.
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
milanmajchrak
left a comment
There was a problem hiding this comment.
Check the comment
…display of bundle controls in correct column
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
|
Fixed the table to break words, that are too long, looks like this now: |
There was a problem hiding this comment.
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