• 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 have read, understood, and agreed to those terms.
Issue Details (XML | Word | Printable)

Key: VWR-13743
Type: New Feature New Feature
Status: Open Open
Priority: Normal Normal
Assignee: Unassigned
Reporter: Soft Linden
Votes: 8
Watchers: 5
Operations

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

Change Request: Change double click action for inventory wearables

Created: 28/May/09 10:13 AM   Updated: 29/Sep/09 09:27 AM
Return to search
Component/s: User Interface
Affects Version/s: 1.20, 1.21, 1.22, 1.23 Release Candidate
Fix Version/s: None

File Attachments: 1. File slviewer-0-v11905-InventoryDoubleClickActions_v2.patch.bz2 (0.7 kB)

Issue Links:
Relates

Last Triaged: 10/Sep/09 10:13 AM
Linden Lab Issue ID: DEV-33391
Patch attached: Patch attached


 Description  « Hide
Carl Wilder's writeup from VWR-1825:

i really think this issue is worth addressing, with a slight modification that would mitigate for those who are concerned with compatibility.

First, I think any change to "double-click" behavior should also have a checkbox in Preferences that switches between the current behavior and any new behaviors. The default(s) can be for current behavior so that existing users are not confused.

Second, any argument about consist UI should note that as of viewer 1.23 the pie menus have been considerably reworked, which certanly broke MY muscle-memory and has created a precedent for UI changes such as the one proposed here.

Finally, of all the double-click behaviors the one that seems the most "incorrect" today is double-clicking an object. Double-click is a shortcut for a common action, but seriously how often does anyone want to look at object properties?

With the proliferation of attachments I would argue that the MOST common use of objects (apart from rezzing, which cant be done with double-click anyway) is "wear".

I therefore propose:

(a) A new Preferences checkbox that says "Double click object to wear"
(b) This preference is OFF by default.
(c) When OFF the current (display properties) behavior is active.
(d) When ON, double clicking an object will wear the object.

As a side note, the WORST that could happen with a spurious double click is that an unusual object ends up being worn for a few seconds (and I've seen that happen enough anyway). Therefore this proposed change has no deleterious effects.



 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
Soft Linden made changes - 28/May/09 10:13 AM
Field Original Value New Value
Link This issue Relates to VWR-1825 [ VWR-1825 ]
Henri Beauchamp added a comment - 30/May/09 02:41 AM - edited
I propose that not only the double-click action on objects be changed for "wearing" them (when not yet attached), but also to detach already attached objects.

This also could be usefully extended to removing (un-wearing) wearable items (clothes) when they are already worn (complementary action to the existing action for wearable items).

Here is a patch which was designed for v1.19.0.5 but still applies to v1.22 and v1.23RC2 viewers.

It has been used in the Cool SL Viewer since its first release (1.18.4.3) and nobody ever complained about it, much to the contrary... This is one of those features you cannot do without any more once you tried it.


Henri Beauchamp made changes - 30/May/09 02:41 AM
Attachment slviewer-0-v11905-InventoryDoubleClickActions_v2.patch.bz2 [ 24801 ]
Affects Version/s 1.22 [ 10430 ]
Affects Version/s 1.21 [ 10370 ]
Affects Version/s 1.20 [ 10350 ]
Affects Version/s 1.23 Release Candidate [ 10433 ]
Patch attached [Patch attached]
Geneko Nemeth made changes - 01/Jun/09 05:40 PM
Summary Change Request: Change double click action for objects Change Request: Change double click action for inventory wearables
Silver Key added a comment - 04/Jun/09 06:27 AM
This would be a very good UI improvement.

lindenrobot added a comment - 04/Jun/09 10:13 AM
Automated note to patch contributor: if you haven't done so already, please make sure you have a contribution agreement on file and that your patch includes an edit to doc/contributions.txt properly crediting yourself for this patch. See https://wiki.secondlife.com/wiki/Submitting_code for more details.

lindenrobot made changes - 04/Jun/09 10:13 AM
Last Triaged 04/Jun/09 10:13 AM
Linden Lab Issue ID DEV-33391
Ellla McMahon made changes - 22/Jul/09 03:15 PM
Link This issue is related to by SNOW-130 [ SNOW-130 ]
Alexa Linden made changes - 10/Sep/09 10:23 AM
Last Triaged 04/Jun/09 10:13 AM 10/Sep/09 10:13 AM
Ezian Ecksol made changes - 29/Sep/09 09:26 AM
Link This issue is original of duplicate VWR-15426 [ VWR-15426 ]
Ezian Ecksol made changes - 29/Sep/09 09:27 AM
Link This issue is original of duplicate VWR-15426 [ VWR-15426 ]