link tool lookup window should open to link reference's container

Bug #237103 reported by Andy Altepeter
2
Affects Status Importance Assigned to Milestone
Silva
Fix Released
Wishlist
Unassigned

Bug Description

If an existing link in kupu goes to an area within Silva, it would be neat if the "get link reference" button in the link tool opened the lookup window to the container of the existing link.

Use case: I accidentally selected the wrong link, in the same container as the correct link, and that container took five clicks in the lookup window to get to.

(this may happen more frequently now that clicking the radio button places the link)

Revision history for this message
Eric Casteleijn (thisfred) wrote :

Hmm, I thought we had this solved. I know we do in DLCMS, so it should be ported to silva core if it isn't already there. Assigning to my bad self...

Changed in silva:
assignee: nobody → thisfred
Revision history for this message
Andy Altepeter (aaltepet) wrote :

I should point out that this "lack of a feature" was in Silva 2.0, so it is quite possible that the trunk is working correctly.

Changed in silva:
milestone: none → 2.1
Changed in silva:
assignee: thisfred → wim-boucquaert
Changed in silva:
assignee: wim-boucquaert → aaltepet
Revision history for this message
Andy Altepeter (aaltepet) wrote :

this is actually rather difficult. how do you determine whether a link is a valid path, and from that how you know what the container is? It may be possible to send the existing link as a parameter, and then, the first thing the lookup window does is determine if the existing link is a path to a silva object, get it's container, and redirect.

Revision history for this message
Kit Blake (kitblake) wrote :

The lookup window is probably the most complex screen in Silva. This adds yet more complexity but it's a great feature.

It should be possible to check whether the linked object exists, then get it's container, and use that path as the url. Or maybe check whether the linked object is a container, and if not, go to its container. This could handle the not found use case, and bring the user to the container where the link destination used to be.

Of course, the enclosing container could be not found too, so I guess we should check whether it exists, and if not, just bring up the lookup window in the current container.

Revision history for this message
Andy Altepeter (aaltepet) wrote :

this is now implemented and checked into the Silva trunk (to be 2.2)

Changed in silva:
assignee: Andy Altepeter (aaltepet) → nobody
importance: Undecided → Wishlist
milestone: 2.1 → 2.2
status: New → Fix Committed
Changed in silva:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.