elcsiga commented on PR #9109:
URL: https://github.com/apache/nifi/pull/9109#issuecomment-2270990627

   Comparing different versions looks great, version selection and filters are 
working well 👍 
   
   <img width="733" alt="image" 
src="https://github.com/user-attachments/assets/57b234ad-abb7-448a-ad45-937ef5b26624";>
   
   ### Comments about version selection, maybe related to this PR:
   
   - for the 1st glance meaning of the 2 numbers in the 2 dropdowns wasn't 
super clear for me. Maybe a label for the dropdowns would be helpful.
   - version selection in the 'Change Version' dialog is a bit more 
user-friendly, having comments and dates. I'm not sure is there anything to do 
here without redesigning the header, since there is no place to display 
additional information in the dropdowns. And user can also use 'Change Version' 
dialog to select. 
   <img width="403" alt="image" 
src="https://github.com/user-attachments/assets/4cf40fa1-01de-4f88-b31e-3916eaaceb33";>
   
   ### A strange issue:
   
   Workflow 1:
   
   1. Select version 1 in the 'Change Version' dialog
   2. Click the 'compare' icon of version 3. Compare dialog will show 
comparison between 3 and 3. 
   <img width="671" alt="image" 
src="https://github.com/user-attachments/assets/6d8f50cf-4298-412f-b138-733da16357c3";>
   
   Workflow 2:
   
   1. Select version 3 in the 'Change Version' dialog
   2. Click the 'compare' icon of version 2. Compare dialog will correctly show 
comparison between 3 and 3
   3. Select version 3 in the 'Change Version' dialog  
   4. Click the 'compare' icon of version 3. Compare dialog will show 
comparison between 3 and 3, but with wrong content:
   <img width="682" alt="image" 
src="https://github.com/user-attachments/assets/6a88482d-53bb-42d7-a80d-1a6af498f4ab";>
   
   
   
   
   
   
   
   


-- 
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]

Reply via email to