Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001096LuxRenderLuxGUIpublic2011-06-10 09:412012-09-11 06:10
ReporterJ the Ninja 
Assigned ToLord Crc 
PrioritylowSeveritytextReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version1.0Fixed in Version1.0RC4 
Summary0001096: Eliminate "yes/no" labels in "stop render and start a new one" dialogue
DescriptionWhen opening a scene file while another render is in progress, LuxRender will pop up a dialogue asking the user if they would like to stop the current render and start a new one. However, the buttons are labeled "yes" and "no". They should be something more descriptive that does not require the user to carefully read the dialogue, such as "Start new" and "Continue current".

A screenshot of the offending dialogue is attached.
TagsNo tags attached.
Mercurial Changeset #
Requires Documentation UpdateNo
Requires Exporter Update
Attached Filespng file icon Screen shot 2011-06-10 at 9.30.30 AM.png [^] (25,792 bytes) 2011-06-10 09:41


diff file icon A.diff [^] (1,128 bytes) 2012-01-15 16:18 [Show Content]
diff file icon B.diff [^] (671 bytes) 2012-01-15 16:18 [Show Content]

- Relationships

-  Notes
(0003298)
Omniflux (developer)
2012-01-15 16:20

Patch A implements the change as requested.

Patch B simplifies the current prompt and is my personal preference.
(0003299)
J the Ninja (developer)
2012-01-15 23:04

Patch B has yes/no dialog boxes still. That's the crux of the issue, labeling buttons with "Yes" and "No" is rarely a good idea. It's better to label them with an action verb of what clicking said button will do.
(0003300)
Omniflux (developer)
2012-01-15 23:23

I submitted both ways because I do not like the phrasing in Patch B and could not think of any that looked right. What do you think of changing the message to

'Loading a new scene will halt rendering of the current scene. Do you wish to continue?'

and using 'Load new scene' for the first button and 'Cancel' for the second?
(0003302)
Lord Crc (administrator)
2012-01-17 15:45

Thanks for the patch. Based on your patch I created a slightly more general function, and implemented the change using this. I think we could use it to improve some of our other dialogs as well.

I also changed the wording slightly, feedback welcome :)
(0003340)
J the Ninja (developer)
2012-01-28 09:46

New dialog looks great. :)

- Issue History
Date Modified Username Field Change
2011-06-10 09:41 J the Ninja New Issue
2011-06-10 09:41 J the Ninja File Added: Screen shot 2011-06-10 at 9.30.30 AM.png
2011-08-31 05:17 jeanphi Target Version => 1.0
2012-01-15 16:18 Omniflux File Added: 0001096-A.diff
2012-01-15 16:18 Omniflux File Added: 0001096-B.diff
2012-01-15 16:20 Omniflux Note Added: 0003298
2012-01-15 23:04 J the Ninja Note Added: 0003299
2012-01-15 23:23 Omniflux Note Added: 0003300
2012-01-17 15:45 Lord Crc Note Added: 0003302
2012-01-17 15:45 Lord Crc Assigned To => Lord Crc
2012-01-17 15:45 Lord Crc Status new => feedback
2012-01-28 09:46 J the Ninja Note Added: 0003340
2012-01-28 09:46 J the Ninja Status feedback => assigned
2012-01-28 09:46 J the Ninja Status assigned => resolved
2012-01-28 09:46 J the Ninja Resolution open => fixed
2012-08-21 06:58 jeanphi Fixed in Version => 1.0RC4
2012-09-11 06:10 jeanphi Status resolved => closed


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker