devabhishekpal opened a new pull request, #7667:
URL: https://github.com/apache/ozone/pull/7667

   ## What changes were proposed in this pull request?
   HDDS-12043. Mark fixed column with disabled checkbox.
   
   Please describe your PR in detail:
   * In MultiSelect component, we pass a `fixedColumn` prop which decides the 
menu option which should not be selectable. This is useful to make table have 
at-least one column fixed even if other columns are deselected.
   * But in the current implementation this disabled option is rendered with an 
active checkbox causing the confusion that the option is selectable.
   * This PR modifies the option passed to have isDisabled explicitly set for 
fixedColumn and use that as the options to the menu.
   * This will render the fixedColumn with a grey disabled checkbox, which 
provides proper feedback to the user that the column is not selected/fixed.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-12043
   
   
   ## How was this patch tested?
   Patch was tested manually
   Before the changes:
   <img width="275" alt="Screenshot 2025-01-08 at 20 54 20" 
src="https://github.com/user-attachments/assets/eb941f21-d1cb-493a-8b0e-dcdf262ab560";
 />
   
   After the changes:
   <img width="215" alt="Screenshot 2025-01-08 at 20 54 56" 
src="https://github.com/user-attachments/assets/5b99a16a-f87c-490f-9784-2fb49ab7185c";
 />
   
   Same has been tested out for the Pipelines page, Volumes page and Containers 
page i.e wherever fixedColumn is being passed to MultiSelect


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to