U4-433 - Insert/edit link - Media items are dimmed

See attached dump.

''Originally created on CodePlex by [mikkeljohansen|http://www.codeplex.com/site/users/view/mikkeljohansen]'' on 9/15/2011 11:50:49 PM [Codeplex ID: 30477 - Codeplex Votes: 7]

Imported comments

''Comment by [ovalsquare|http://www.codeplex.com/site/users/view/ovalsquare] on 11/2/2011 2:18:53 AM:'' I have verified that this is the case (4.7). I've ignored it until now when a client has pointed it out as a frustration.

Unfortunately, after looking through the source, I am unable to determine where the class dim is mistakenly applied.

''Comment by [PeterD|http://www.codeplex.com/site/users/view/PeterD] on 7/10/2012 10:50:29 AM:'' I came across this as well. It is caused because the items are in a folder. The folder has no link itself, therefor the code dims the item out (hence all children are dimmed as well)

If you click the folder, it returns a 404 instead of opening the folder. To open the folder, you explicitly have to click the small triangle. I created a patch for this, let me try to create a pull request.

In short: I edited presentation\umbraco\trees\basemediatree.cs and added a couple of lines:

if (this.DialogMode == TreeDialogModes.fulllink) { string nodeLink = GetLinkValue(dd, dd.Id.ToString()); if (!String.IsNullOrEmpty(nodeLink)) { xNode.Action = "javascript:openMedia('" + nodeLink + "');"; } else { //edited part if (dd.ContentType.Alias.ToLower() == "folder") { xNode.Action = "javascript:jQuery('.umbTree #"+dd.Id.ToString()+"').click();"; } else { xNode.Action = null; xNode.Style.DimNode(); } //end edited part } }

''Comment by [PeterD|http://www.codeplex.com/site/users/view/PeterD] on 7/10/2012 10:29:56 PM:'' In addition: this issue is related to #29003 which mentions the resulting 404 if you click on a folder.

''Comment by [sebastiaan|http://www.codeplex.com/site/users/view/sebastiaan] on 7/25/2012 5:00:55 PM:'' Fixed by PeterD and will be released with 4.9.0

** Closed by sebastiaan 07/25/2012 8:00AM

''Comment by [sebastiaan|http://www.codeplex.com/site/users/view/sebastiaan] on 7/26/2012 7:35:24 AM:'' re-opening to change status correctly :)

Comments