When using a page or block display for a view, there is a contextual link for editing that view directly.

But when a view is used as a panel pane, you have to edit the panel itself, find the name of the view, and go edit the view directly.

CommentFileSizeAuthor
#1 ctools-1669756-1.patch1.24 KBdawehner

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs review
StatusFileSize
new ctools-1669756-1.patch1.24 KB

This adds support for content panes displays, not sure what to do with the other ways.

azinck’s picture

azinck commented

Works for me. Thanks so much; it would be nice to get these to work on any display type but this is a definite step in the right direction.

FYI, here's a related discussion: #1461870: Contextual links (cogs) for my views panes are missing (theme dependent)

pixelwhip’s picture

pixelwhip commented
Status: Needs review » Reviewed & tested by the community

Patch in #1 worked for me as well. Saves a lot clicking!

Lowell’s picture

Lowell commented

Works as described, ctools 7.x-1.3

however, does not work when using a panel pane in a variant of the Node Template panel page.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes

This is related to #1948278: Get an edit link for the view panes in Page Manger.

saltednut’s picture

saltednut
she/her
Primary language English
Location Chicago
commented

Its 2014 and this patch still works. RTBC++

gmclelland’s picture

gmclelland commented

Are we sure this is still needed? I'm using the latest ctools and Panels modules-dev versions. I'm seeing the links to edit the view in the html on my Panels pages that have a view's content pane on them.

The problem I'm seeing is that the contextual link isn't visible. It gets hidden behind other contextual links. Using css, you can move the cog over and see the links. I believe there is a core issue somewhere about fixing contextual links so they don't overlap. This might be the issue #1914966: Nested contextual links triggers don't work well

azinck’s picture

azinck commented

I haven't tested this lately; maybe a fix for this made its way in independently. But I can confirm at the time I tested this that 1) I was aware of the problem of overlapping cogs and 2) this was beyond the problem of overlapping cogs. The links were definitely actually missing. But again, maybe it's been independently fixed. Will see if I can take a look with the latest dev versions.

raulmuroc’s picture

raulmuroc commented

Yes, this patch is still RTBC. I had this problem right now and found this issue. Applied the patch to latest Ctools and now it works so this patch still has to be committed to the -dev.

gmclelland’s picture

gmclelland commented

@RaulMuroc - before you patched, were you using the latest dev versions of Ctools and Panels?

betoscopio’s picture

betoscopio
Primary language Spanish
commented

For may case, the patched applied. And views seems to work ok with panels.
But after some tests, and using the ctools-1.5 without this patch, panels-1.4 and dev version of draggableviews, seems to work without problem to me.

japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Fixed

Looks good to me, applied.

  • japerry committed f7fb6eb on 7.x-1.x authored by dawehner 
    Issue #1669756 by dawehner, gmclelland, tim.plunkett: Provide a...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

I am having this problem today, on a site with CTools with this patch incorporated.

So I figured out how to fix it, and filed this follow-up issue:
#2506615: Contextual links still not showing for Content pane views in Panels
(along with a quick one-line patch).