Viewing 11 replies - 16 through 26 (of 26 total)
  • Thread Starter Henry Wright

    (@henrywright)

    Yeah that is the downside. That said, the processing is done on the client so it shouldn’t have an impact on site performance.

    I’ve been on Stack Overflow looking for a cleaner solution. input seems the only event that will do what’s needed?

    Thread Starter Henry Wright

    (@henrywright)

    I’ve been doing some more digging with reference getting this working.

    .on('paste', function (e) {
        e.preventDefault();
        var contentOnBlur = (e.originalEvent || e).clipboardData.getData('text/plain') || prompt('Paste something..');
        contentOnBlur = contentOnBlur.replace(/(<([^>]+)>)/ig,'');
        document.execCommand('insertText', false, contentOnBlur);
    })

    The paste event works nicely in Chrome although using the above snippet results in not being able to paste anything in to the title when using IE.

    Somebody has posted a solution to this exact IE problem on Stack Overflow:
    https://stackoverflow.com/questions/2176861/javascript-get-clipboard-data-on-paste-event-cross-browser/6804718#6804718

    PHP I can handle but I’m out of my depth with JavaScript. It might make more sense to you ??

    Plugin Author Ella

    (@ellatrix)

    Paste and execCommand are difficult to get working cross browser. That’s why I prefer to filter after it’s been pasted. If you want to filter during the past process, it’s better to use TinyMCE for that. I used that to convert links to embeds with oEmbed.

    Plugin Author Ella

    (@ellatrix)

    But I don’t think it’s work to create another instance just for the title…

    Thread Starter Henry Wright

    (@henrywright)

    Paste and execCommand are difficult to get working cross browser.

    Agreed! I spent a good few hours realising that yesterday ??

    If you want to filter during the past process, it’s better to use TinyMCE for that.

    Do you mean by creating a new instance? At first I thought it would be overkill to create a new instance for just the title, but as you say maybe it is a good way to ensure cross-browser and WP theme compatibility going forward.

    e.g. It would resolve:
    https://www.ads-software.com/support/topic/title-not-editable-using-certain-themes

    Plugin Author Ella

    (@ellatrix)

    Why would that resolve it?

    Thread Starter Henry Wright

    (@henrywright)

    Sorry, I should have said *could* resolve.

    The content (handled by tinyMCE) is editable and working as expected. It’s just the title which doesn’t seem to be editable – leading me to think that a tinyMCE instance for the title could resolve some of the cross-browser issues seen so far.

    Plugin Author Ella

    (@ellatrix)

    TinyMCE only provides tools to edit the field. Apart from that both fields are exactly the same. The reason the title field is not editable lies somewhere else.

    Thread Starter Henry Wright

    (@henrywright)

    The reason the title field is not editable lies somewhere else.

    Yeah, I think it’s the strip tags functionality that is causing it to be not editable as the problem wasn’t there (was it?) before that functionality was introduced.

    TinyMCE provides a way of stripping tags when initialising e.g

    paste_as_text: true,

    Plugin Author Ella

    (@ellatrix)

    No, that’s not the problem either. The problem is the way the plugin filters the title and the content. The theme may have multiple main loop which may use the title which the plugin filters. The plugin will only filter it once, causing the next time it uses the title (the one that’s visible for example) to be uneditable. I’m working on it, but it may not be fully solvable.

    Thread Starter Henry Wright

    (@henrywright)

    Ah, OK – that explains it.

    I suppose if the problem was the strip tags functionality then all themes would be affected – which isn’t the case.

Viewing 11 replies - 16 through 26 (of 26 total)
  • The topic ‘Copying then pasting into title’ is closed to new replies.