Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1258)

Unified Diff: chrome/browser/resources/shared/js/cr/ui/card_slider.js

Issue 9358073: First version of the time slicing on the urls. Not ready for review yet. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added relative time to time slicing Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/shared/js/cr/ui/card_slider.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/card_slider.js b/chrome/browser/resources/shared/js/cr/ui/card_slider.js
index f7da9aa002afd9229f44fa44e1da20302f8da5dc..cb04a3f14ce2971cfccccd5cd74c2b0b4747cb37 100644
--- a/chrome/browser/resources/shared/js/cr/ui/card_slider.js
+++ b/chrome/browser/resources/shared/js/cr/ui/card_slider.js
@@ -344,7 +344,7 @@ cr.define('cr.ui', function() {
if (this.currentCard_ == -1)
this.currentCard_ = 0;
else if (index <= this.currentCard_)
- this.selectCard(this.currentCard_ + 1, false, true);
+ this.selectCard(this.currentCard_ + 1, false, true, true);
this.fireAddedEvent_(card, index);
},
@@ -436,14 +436,22 @@ cr.define('cr.ui', function() {
* current position to new position.
* @param {boolean=} opt_dontNotify If true, don't tell subscribers that
* we've changed cards.
+ * @param {boolean=} opt_forceChange If true, ignore if the card already
+ * selected.
*/
- selectCard: function(newCardIndex, opt_animate, opt_dontNotify) {
+ selectCard: function(newCardIndex,
+ opt_animate,
+ opt_dontNotify,
+ opt_forceChange) {
this.assertValidIndex_(newCardIndex);
var previousCard = this.currentCardValue;
var isChangingCard =
!this.cards_[newCardIndex].classList.contains('selected-card');
+ if (typeof opt_forceChange != 'undefined' && opt_forceChange)
+ isChangingCard = true;
+
if (isChangingCard) {
if (previousCard)
previousCard.classList.remove('selected-card');
« no previous file with comments | « chrome/browser/resources/ntp4/suggestions_page.js ('k') | chrome/browser/ui/webui/ntp/new_tab_page_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698