{"action":"created","issue":{"url":"https://api.github.com/repos/Slicer/Slicer/issues/636","repository_url":"https://api.github.com/repos/Slicer/Slicer","labels_url":"https://api.github.com/repos/Slicer/Slicer/issues/636/labels{/name}","comments_url":"https://api.github.com/repos/Slicer/Slicer/issues/636/comments","events_url":"https://api.github.com/repos/Slicer/Slicer/issues/636/events","html_url":"https://github.com/Slicer/Slicer/pull/636","id":194715448,"number":636,"title":"ENH: Implemented DICOMDetailsPopup as window and dialog","user":{"login":"che85","id":10195822,"avatar_url":"https://avatars.githubusercontent.com/u/10195822?v=3","gravatar_id":"","url":"https://api.github.com/users/che85","html_url":"https://github.com/che85","followers_url":"https://api.github.com/users/che85/followers","following_url":"https://api.github.com/users/che85/following{/other_user}","gists_url":"https://api.github.com/users/che85/gists{/gist_id}","starred_url":"https://api.github.com/users/che85/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/che85/subscriptions","organizations_url":"https://api.github.com/users/che85/orgs","repos_url":"https://api.github.com/users/che85/repos","events_url":"https://api.github.com/users/che85/events{/privacy}","received_events_url":"https://api.github.com/users/che85/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-12-09T22:45:28Z","updated_at":"2016-12-09T23:04:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Slicer/Slicer/pulls/636","html_url":"https://github.com/Slicer/Slicer/pull/636","diff_url":"https://github.com/Slicer/Slicer/pull/636.diff","patch_url":"https://github.com/Slicer/Slicer/pull/636.patch"},"body":"- default: DICOMDetailsDialog which is bound to the mainWindow and slides in from top\r\n  - closing with esc (which could be improved)\r\n- when using DICOMDetailsWindow, it always says on top\r\n  - resize/move event are overridden called and saves the dimensions to the settings for next call\r\n- old ones like DICOMDetailsPopup and DICOMDetailsDock should be removed since they don't improve any usability\r\n\r\n![image](https://cloud.githubusercontent.com/assets/10195822/21067106/3c0d62ee-be37-11e6-818e-f61666dd2fe0.png)\r\n"},"comment":{"url":"https://api.github.com/repos/Slicer/Slicer/issues/comments/266147383","html_url":"https://github.com/Slicer/Slicer/pull/636#issuecomment-266147383","issue_url":"https://api.github.com/repos/Slicer/Slicer/issues/636","id":266147383,"user":{"login":"fedorov","id":313942,"avatar_url":"https://avatars.githubusercontent.com/u/313942?v=3","gravatar_id":"","url":"https://api.github.com/users/fedorov","html_url":"https://github.com/fedorov","followers_url":"https://api.github.com/users/fedorov/followers","following_url":"https://api.github.com/users/fedorov/following{/other_user}","gists_url":"https://api.github.com/users/fedorov/gists{/gist_id}","starred_url":"https://api.github.com/users/fedorov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fedorov/subscriptions","organizations_url":"https://api.github.com/users/fedorov/orgs","repos_url":"https://api.github.com/users/fedorov/repos","events_url":"https://api.github.com/users/fedorov/events{/privacy}","received_events_url":"https://api.github.com/users/fedorov/received_events","type":"User","site_admin":false},"created_at":"2016-12-09T23:04:27Z","updated_at":"2016-12-09T23:04:27Z","body":"@pieper I don't know if those things still work, but as a comment, I have not seen radiology reading workstations that would optimize DICOM interface UI for utilization of second monitor (I also admit I probably didn't see too many). I am curious what is the proportion of Slicer users who find separate window for DICOM browser feature useful. But of course we will never know the answer to that question.\r\n\r\n@che85 for the sake of efficiency, we should probably just proceed optimizing DICOM workflow in our extensions as we discussed earlier. Features of the main application are optimized to accommodate a broad range of usage scenarios, and we really don't have much data to back choices made for one reason or another. Fortunately, we have sufficient control to optimize individual modules to support our target users, and implement design choices as we see fit, and refine based on the direct feedback from the users we interact with."}}
{"actor":{"display_login":"fedorov"}}