Opened 13 years ago

Closed 7 years ago

#548 closed New Feature (wontfix)

Context menu should recursively allow user to edit parent's Tags

Reported by: dricks2222 Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.5
Keywords: HasPatch Cc:

Description (last modified by Alfonso Martínez de Lizarrondo)

If you have :

<P>
  <DIV>
    <TABLE>
      <TR>
        <TD>
        Cursor here
        </TD>
      </TR>
    </TABLE>
  </DIV>
</P>

And you right click where 'Cursor here' is, the context menu should show :


Edit Table Cell
Edit Table Row
Edit Table
Edit DIV
Edit P


All the hierarchy until you reach the <BODY> Tag. Else, how are you suppose to edit the style of a the P tag for instance? ...

Attachments (1)

fckplugin.js (10.2 KB) - added by dricks2222 12 years ago.
nodePath modified plugin

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by dricks2222

Sorry, i missed the Wiki formatting. So, the menu should be :

Edit Table Cell
Edit Table Row
Edit Table
Edit DIV
Edit P

comment:2 Changed 13 years ago by Alfonso Martínez de Lizarrondo

Description: modified (diff)

Having the full hierarchy of DOM nodes show up in the context menu sounds like a bad idea. After all it's a context menu, so it should show things relative only to the context, and everything beyond the table isn't needed:

if you want to edit the div or p (supposing that there are the proper commands for those tags) should be available when you right click on that element, or you would end up with a context menu really too long as soon as some elements get nested.

There are other options like having a status bar with all the nodes (available as a plugin) that are better suited if you want to show all the hierarchy anyway.

Voting for WontFix.

comment:3 Changed 13 years ago by Frederico Caldeira Knabben

I also see the "Elements Path Bar" as a better option, but I'm sure others would also enjoy the context menu thing. I don't think it would be a core feature. I see it as a plugin instead.

There is no bad on leaving this ticket open, even if we'll not be working on it right now. It could motivate others to contribute this plugin.

comment:4 Changed 12 years ago by dricks2222

I used the nodePath plugin to start with.
I changed it so you can now change ID and style of every editable tag, as well as the innerHTML (when available).

You have to change the NodePathImg variable in the js to have a correct img button. I haven't been able to find in which CSS the 'WYSIWYGPfad' have to be, so styles are hard-coded. Works pretty well both on IE and FFx. Not tested with other navigators. See attachment for plugin.

Changed 12 years ago by dricks2222

Attachment: fckplugin.js added

nodePath modified plugin

comment:5 Changed 12 years ago by Wojciech Olchawa

Keywords: HasPatch added

comment:6 Changed 9 years ago by Wiktor Walc

Version: FCKeditor 2.5

comment:7 Changed 7 years ago by Jakub Ś

Resolution: wontfix
Status: newclosed

As other have noticed it - elements path is perfect for this approach.
I see one request plus one plugin for FCKeditor.

Taking the above into account and knowing that we won't implement it as core feature I closing this issue as won't fix.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy