Ticket #9161 (closed: duplicate)

Opened 7 years ago

Last modified 5 years ago

Mark output ws properties with existing ws names in algorithm dialogs

Reported by: Arturs Bekasovs Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.2
Component: GUI Keywords:
Cc: peter.parker@… Blocked By:
Blocking: Tester: Peter Peterson

Description

Some users were complaining about the fact that currently it's too easy to accidentally overwrite a workspace when running algorithms using the GUI.

The most obvious solution here would be to show confirmations when we are about to overwrite something. However, in situations when we are experimenting with algorithm parameters and don't really care about whether previous output is overwritten, this can easily become irritating.

That's why, the proposed solution is as follows:

If the output workspace field references the workspace which already exists in the ADS, we mark that field somehow (e.g. change it's background color) so that the user is aware that this might cause data loss.

It should not be too bold to make user think that he is doing something wrong, yet noticeable enough to make people consider consequences before hitting Run.

Peter is currently doing lots of changes around that area, though we've decided that these ones are independent enough for me to be able to implement them myself.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

Yes they are indenpendant enough, but you need to discuss with peter how the approaches work, so that the result is aesthetically pleasing, and not a confusing dogs dinner of visual effects.

comment:2 Changed 7 years ago by Arturs Bekasovs

  • Status changed from assigned to verify
  • Resolution set to duplicate

In the end it was decided that it makes more sense for Peter to implement this as a part of #2480. Sorry for wasting everyone's time.

comment:3 Changed 7 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:4 Changed 7 years ago by Peter Peterson

  • Status changed from verifying to closed

I agree that the other ticket will cover this well enough.

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10004

Note: See TracTickets for help on using tickets.