2017-02-02 3 views
0

私はこれをランダムなクォートジェネレータであるcodePen持っています。 私はまた、生成された見積もりをつぶすべきtweetボタンを持っています。 現在、それを押すと、表示されている現在の見積もりではなく、別のランダムな見積もりがつぶれます。ここでは関数は次のとおりです。ボタンが適切にtweetingしていないテキスト

$(".twitter-share-button").click(function(){ 
    $(this).attr("href", 'https://twitter.com/intent/tweet?text=' + randomNumber); 
    }); 

function tweetIt(){ 
var randomNumber = Math.floor(Math.random() * (quotes.length)); 
window.open("https://twitter.com/intent/tweet?text=" + quotes[randomNumber]); 
} 

私は引用符が無駄にだけにある現在のIDを選択することでこれを解決しようとした:

function tweetIt(){ 
    var tweet = document.getElementById('quote').value; 
    window.open("https://twitter.com/intent/tweet?text=" + quotes[randomNumber]); 
}; 

これで任意の助けいただければ幸いです。

答えて

0

あなたが「quoteDisplay」のIDを持つ要素の中にあなたの現在の見積書を格納している:

function newQuote() { 
    var randomNumber = Math.floor(Math.random() * (quotes.length)); 
    // The content for your tweet is being set here 
    document.getElementById('quoteDisplay').innerHTML = quotes[randomNumber]; 
}; 

あなたは現在の見積もりをつぶやきたいのであれば、あなたがその特定の要素のtextContentプロパティをターゲットにする必要があります:

function tweetIt(){ 
    // Get the contents of your current tweet 
    var tweet = document.getElementById('quoteDisplay').textContent; 
    // Open a new window with that tweet content 
    window.open("https://twitter.com/intent/tweet?text=" + tweet); 
}; 
+0

「innerText」は使用しないでください。それは標準ではなく、FireFoxには存在しません。 'textContent'を使用してください。 – Barmar

+0

これを明確に説明していただきありがとうございます。また、@ Barmar – YoungCoder

+0

@ Barmarありがとう、それは私も答えを調整していただきありがとうございます。 –

関連する問題