search

Ptr32Void / OSTrICa

306 stars 92 forks source link

Requirements file for ease of installing #5

Closed PidgeyL closed 8 years ago

PidgeyL commented 8 years ago
Ptr32Void commented 8 years ago

Hi,

Thanks for your update. I added the requirements into the master branch and updated changelog.txt.

Regarding the "Zoom fix", I understand why you want to modify the zoom option but if you have lots of IoCs (or intelligence data) which are split in multiple clusters (data is not related to each other) you won't be able to see them all. I think it would be beneficial for the analyst to see all the nodes/clusters available.

If for some reason I misread the pull request related to the Zoom please let me know.

Closing now.

PidgeyL commented 8 years ago

The problem with the zoom is that, when you scroll out too far, you lose track of your data. Maybe a reset button would be an alternative, but it happened often to me

Ptr32Void commented 8 years ago

Yes, I know, is an issue. Unfortunately your fix might bring other issues in the code, that's why I did not merge it. But, yes I agree with you; a "reset button" would be nice. If you plan to add it, just open an issue here and then I will merge it; otherwise I will add it ASAP.

Thanks for contributing to the project.