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-446
Type: New Feature New Feature
Status: Resolved Resolved
Resolution: Fixed
Priority: Low Low
Assignee: Zen Linden
Reporter: Mr Greggan
Votes: 13
Watchers: 0
Operations

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

Automatically start renaming new user-created assets and automatically select new user-created folders

Created: 15/Apr/07 12:55 AM   Updated: 04/Jul/07 07:21 PM
Component/s: User Interface, Inventory
Affects Version/s: 1.14.0.x, 1.15.0.x, Source code
Fix Version/s: 1.17.1

File Attachments: 1. Text File select_newlyCreatedFolders_AND_automaticallyStartRenaming_allNewlyCreatedAssets.patch (5 kb)

Issue Links:
Relates
 

Source Version: 1.15.0.2
Linden Lab Issue ID: SL-45064
Patch attached: Patch attached


 Description  « Hide
When a new folder is created in the inventory, the first thing you'll want to do is rename it. Instead, you have to go find it, right-click, choose rename, then type the new name.

It would be much more intuitive if the new folder is automatically selected, with the cursor in the name field ready to type. Or a pop up on the creation of a new folder so you can specify name and location before it's created.


 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
Still Defiant - 04/May/07 07:27 AM
I hope you don't mind me editing this feature request, Mr Greggan; I came to propose a feature request that is similar to your initial request, yet adds a little more. I figured it would be better to build on your issue rather than start a new one.

Here is a proposed patch (only tested on GNU/Linux) that does two things:

1) immediately selects new user-created folders (From inventory: Create->New Folder OR right-click an existing folder and select 'New Folder'); and

2) immediately begins to rename any new user-created asset (Folder/Script/Note/Gesture/Clothes.../Body Parts...) if it was successfully selected after creation (meaning it was viewable - ie, not hidden by a text filter and not made from the "Recent Items" panel in the case of new folders)

Looking at different ways to implement this, I finally opted to add new member data to LLFolderView because similar data was already present (mNeedsAutoSelect/mAutoSelectOverride), then added the selection callback to LLInventoryView, because it seemed to me to be most contextually relevant there (improvements and/or alternate solutions obviously welcome).

On a side note, Mr Greggan's popup suggestion seems potentially useful as well. It could bypass hidden asset selection/renaming limitations without having to clear filters or force all folder creation to the 'All Items' panel, providing more seemless/consistent behavior (although at some point, I'd rather see newly created empty folders filtered in 'Recent Items', regardless).

Still Defiant - 05/May/07 09:29 AM
Actually, I noticed that this patch also works for new landmarks and assets copied from notecards. That may or may not be good behavior (I'm liking it thought, especially for landmarks).

Still Defiant - 09/May/07 11:05 AM
So patch bugs: mNeedsAutoRename isn't initialized and LLInventoryModel::messageUpdateCore should definitely not be touched for this feature request. Having just discovered LLInventoryCallback's too, I'll rework this and try again later...

Haravikk Mistral - 07/Jun/07 10:48 AM
I'd even go so far as to say that this should be done in-world too with objects. The number of un-named objects out there, and I sympathise as I just want to straight-up build and forget to name things. But if I was prompted I'd think of something and enter it.

Still Defiant - 12/Jun/07 03:55 AM
Because I was recently asked about this issue:

Sorry for not updating this! At this time and in the foreseeable future, I have no intention of reworking the patch or continuing code contributions on JIRA due to personal disagreement with content regulation policy changes in-world. However, sympathizing with users, I want to make clear that a copy of this attached patch is hereby licensed under the same license as the viewer source as of this date, 06/12/07, for anyone wishing to derive and resubmit (also watch for new asset creation behavior from appearance-editing mode). If, for whatever reason, someone needs a full license-code package for such a relatively trivial patch, email me at Still.Defiant@gmail.com.

Also, I like that general idea, Haravikk Mistral.

Torley Linden - 13/Jun/07 10:26 AM
(Removed "[EDIT]" from summary, we know who edited it by viewing this issue's change history.)

I'd really find this useful too... thanks for the patch!