History | Log In     View a printable version of the current page.  
  • All submissions to this site are governed by Second Life Project Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you've read, understood, and agreed to those terms.
Issue Details (XML | Word | Printable)

Key: VWR-1147
Type: New Feature New Feature
Status: Resolved Resolved
Resolution: Fixed
Priority: Normal Normal
Assignee: Sardonyx Linden
Reporter: Fremont Cunningham
Votes: 3
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
1. Second Life Viewer - VWR

A patch set is provided to add an optional 'Confirm Exit' pop-up window for most user client exit methods. Prevents the 'Accidental Quit'.

Created: 09/Jun/07 09:22 PM   Updated: 06/Jul/07 09:00 AM
Component/s: User Interface
Affects Version/s: 1.16.0.x
Fix Version/s: 1.17.3

File Attachments: 1. File Confirm_Exit_Patch.tar (30 kb)

Environment: Source compiling.
Issue Links:
Duplicate
 

Linden Lab Issue ID: SL-44936
Patch attached: Patch attached


 Description  « Hide
Second Life Client Modification: Confirm Exit
=============================================

By Fremont Cunningham July 2007


This is a set of diff files which when applied using the patch command
modify the source files with changes necessary for the new functionality.


The Confirm Exit modification adds an alert panel whenever the window X
button (top-right) is clicked, control-4 is typed, also when the client is
connected if File/Quit is selected or ctrl-q is typed. The alert panel
has 3 buttons: 'Yes', 'Dont ask me again' and 'No'. A user
preference option is added to the Preferences General panel which if
de-selected disables the alert panel, returning the original instant quit
functionality. The option setting is retained between sessions.

The diff/ directory containing the diff files, this file and the patch_ce
script should be placed at the same level as the linden/ directory and the
patch_ce script executed, then the source recompiled.

This patch set was generated for version 1.16.0.5, but may work on later
versions.


 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
Sardonyx Linden - 11/Jun/07 03:47 PM
Hi, Fremont.

Thanks for the patch. Please sign and send in the contribution agreement, as we don't seem to have a copy from you: http://wiki.secondlife.com/wiki/Project:Contribution_Agreement

Also, for future reference: if you could generate a single patch file in unified diff format, instead of many files that are context diffs, that's much easier for us to review work with.

Thank you.

Haravikk Mistral - 13/Jun/07 10:50 AM
This issue is essentially a duplicate of VWR-278 which has more votes, but I'm not sure what to do since it has a patch? Then a Linden /has/ replied to this one, heh, confusing. Both are listed on the small-improvements meta-issue.

Anyways, as for the patch, does it allow for varying modes, for example the option to never ask to confirm on quit, an option to always ask to confirm on quit, and another in-between option to ask to confirm on quit if you have unsaved changes (to a script or notecard for example).

Torley Linden - 13/Jun/07 12:23 PM
Haravikk, great timing -- I happened to be looking for this one just now! We'll resolve VWR-278 as dupe, which makes it no less valid, but focuses attention on this. I'll change internal issue links too. Thank-you!

catherine pfeffer - 16/Jun/07 02:20 PM
Hi,

Thanks for your efforts in making the SL viewer better. But...

I hate software that considers me an idiot and keeps asking
   "Are you _really_ sure you _really_ want to quit?"
Yes, thank you, I really want to quit. Otherwise I wouldn't have clicked on the "close" button...

I know this is a matter of taste. Anyhow... If you add such a feature, it should be possible at least to switch it off in the prefs.

My 2 lindens.

Catherine

Sardonyx Linden - 21/Jun/07 04:16 PM
Applied with some simplifications. Thanks!

catherine: yes, there's an option to dismiss the dialog permanently.

Torley Linden - 06/Jul/07 09:00 AM