Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

report issue #1

Open
tischi opened this issue Dec 4, 2020 · 2 comments
Open

report issue #1

tischi opened this issue Dec 4, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@tischi
Copy link
Owner

tischi commented Dec 4, 2020

@ssgpers
We discussed to be able to report an issue.
It currently looks like this:

Upon right-click at a specific location in BDV:
image

Then this is printed in the log window

### Issue
Location:
(25556.7569085576,19185.080320322304,0.0)
Issue:
This cell is nice!

I next wanted to add a Move to Location option where one can paste the (25556.7569085576,19185.080320322304,0.0) and BDV would move there, allowing to review the issues.

What do you think?

@tischi tischi added the enhancement New feature or request label Dec 4, 2020
@ssgpers
Copy link

ssgpers commented Dec 4, 2020

If I remember correctly, we discuss it in the cotext of making the screenshot of the image together with the issue, right? In this or another way one has to add intormation about which image you are currently looking on.

Another option, hopefully not too difficult, Is to add "Issue" column in a cell based table. Then the messages user puts are stored for each cell and user just sends you back updated table.
The disadvantage of the second approach is that issues can correspond only to cells (or other objects) and not to random locations, but for applications I have in mind this is sufficient

@tischi
Copy link
Owner Author

tischi commented Dec 4, 2020

The image one is looking at is kind of contained in the location, which is the location of the global coordinate system in which the images are presented.

But adding it to the table as well makes sense....

Maybe we could check if there is a segment below the mouse cursor and, if so, a checkbox would appear: [ ] add issue text to table ?!

Should we hardcode the column name? Issues ?

...one could have only one issue per cell in that setup...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants