Page MenuHomePhabricator

[Alpha 5.0.0 (372)] ToC displays identical items
Closed, ResolvedPublic

Description

  1. Go to 'List of most massive black holes' article
  2. ToC is displayed:

IMG_1193.PNG (2×1 px, 152 KB)

NOTE
The article in wiki does not have ToC. Should articles in the apps always display ToC even though the article may have

__NOTOC__

in Page settings?

Acceptance Criteria:

Table of Contents View
Given that the table of contents overlay is visible for article with $title
When I see the first item in the table of contents
Then it should be the $title

Event Timeline

Etonkovidova raised the priority of this task from to Needs Triage.
Etonkovidova updated the task description. (Show Details)
Etonkovidova added subscribers: Aklapper, Etonkovidova.

@Etonkovidova @JMinor the legacy app shows the TOC on this page (see below), so I say we stick w/ that behavior. However, it shouldn't be hard to detect that page property and honor it in the app.

I'll look into a fix for this now.

Screen Shot 2015-10-12 at 4.21.41 PM.png (1×750 px, 436 KB)

@Etonkovidova @JMinor "This is a feature, not a bug." This article actually has a section called "Introduction." We just so happen to also refer to the article's lead section as its "Introduction." So, we can either adopt the legacy app behavior of showing the title instead (see last comment for a screenshot) or something less likely to collide w/ an actual section name.

Android displays ToC without 'Introduction':

'List of most massive black holes' article

Screenshot_2015-10-13-13-12-49.jpg (1×800 px, 391 KB)

To reiterate IRC convo for @JMinor: Android shows the article title instead of "Introduction," and scrolls to top when the title is tapped. In our case, users can also tap the status bar (when visible) to scroll to the top of the article (when ToC isn't visible).

BGerstle-WMF set Security to None.

@Etonkovidova @JMinor I moved this to "Needs Triage" since it needs to be triaged and it's not ready for dev (since no solution is prescribed in the ticket).

Suggest we use the article title. Partly to match Android, but also b/c there are weird potential language and localization issues here.

Checked with 5.0.0.434 on iPhone 5 iOS 9.02

IMG_0016.PNG (1×640 px, 150 KB)

IMG_0014.PNG (1×640 px, 88 KB)

Note: The article title is not different from other ToC items.

Related refinements to ToC happening on T113549