New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 680487 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Jan 2017
Cc:
ntp-dev+bugs@chromium.org
█ ramyan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Feature

Blocking:
issue 683901



Sign in to add a comment

Cleanup: in ClickBasedCategoryRanker "next" is very confusing

Project Member Reported by vitaliii@chromium.org, Jan 12 2017 
I understood that using "next" feels confusing. We are working with an array representation of an NTP, which is vertical. So we could use "above", "below" instead.

Comment 1 by tschumann@chromium.org, Jan 18 2017 

IMO the ranker shouldn't worry too much about the UX representation. We're producing a linear structure of suggestions and 'next' is perfectly fine IMO.
If you want to be picky you can called it "next lower ranked").

That aside, it seems like this is quite a marginal issue which is probably not worth the book-keeping effort of a bug ;-)

Comment 2 by vitaliii@chromium.org, Jan 23 2017

Blocking: -646841

Comment 3 by vitaliii@chromium.org, Jan 23 2017

Blocking: 683901

Comment 4 by vitaliii@chromium.org, Jan 23 2017

Labels: -Pri-2 -M-57 M-58 Pri-3
You may be right.
I will have another look and then decide whether to close.

Comment 5 by vitaliii@chromium.org, Jan 24 2017

Status: WontFix (was: Available)
I had another look, |next| is used only in |OnCategoryDismissed| (4 times) and the current usage seems clear to me.
►

Sign in to add a comment