2017-02-02 9 views
0

У меня есть этот codePen, который является генератором случайных котировок. У меня также есть кнопка твита, которая должна чирикать генерируемую цитату. В настоящее время при нажатии на него он прокручивает другую случайную цитату, а не текущую котировку. Вот функция:Кнопка не правильно чирикает текст

$(".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]); 
} 

Я попытался решить это, выбрав текущий идентификатор котировка в но безрезультатно:

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

Любая помощь с этим было бы весьма признателен.

ответ

0

Вы сохраняете текущую котировку внутри элемента с идентификатором «quoteDisplay»:

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 Спасибо за это, я тоже скорректировал ответ. –