Github user ilgrosso commented on the issue:

    https://github.com/apache/syncope/pull/50
  
    Miscellaneous questions:
    
    1. Why `ConnHistoryDirectoryPanel`, `ConnHistoryConfDirectoryPanel` and 
`ConnHistoryConfMainDirectoryPanel` are separated and not a single class?
    1. Same applies to `ResHistoryDirectoryPanel`, 
`ResHistoryConfDirectoryPanel` and `ResHistoryConfMainDirectoryPanel`
    1. The `Cancel` button is missing in the two history modal windows
    1. There is no option to show the diff with the current configuration
    1. Rather than "reload", the menu item should be "restore" instead
    1. When clicking 'reload', the confirmation message states "Do you really 
want to delete the selected item(s)?"


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to